| Index: test/unittests/compiler/js-typed-lowering-unittest.cc
|
| diff --git a/test/unittests/compiler/js-typed-lowering-unittest.cc b/test/unittests/compiler/js-typed-lowering-unittest.cc
|
| index d4cff090cb0190360321e744e5afecb2638e4d2f..e37d4a2913bf702c386680102ee24f3ff6839bee 100644
|
| --- a/test/unittests/compiler/js-typed-lowering-unittest.cc
|
| +++ b/test/unittests/compiler/js-typed-lowering-unittest.cc
|
| @@ -600,37 +600,34 @@ TEST_F(JSTypedLoweringTest, JSLoadPropertyFromExternalTypedArray) {
|
| NewArrayBuffer(backing_store, sizeof(backing_store));
|
| VectorSlotPair feedback;
|
| TRACED_FOREACH(ExternalArrayType, type, kExternalArrayTypes) {
|
| - TRACED_FOREACH(LanguageMode, language_mode, kLanguageModes) {
|
| - Handle<JSTypedArray> array =
|
| - factory()->NewJSTypedArray(type, buffer, 0, kLength);
|
| - int const element_size = static_cast<int>(array->element_size());
|
| -
|
| - Node* key = Parameter(
|
| - Type::Range(kMinInt / element_size, kMaxInt / element_size, zone()));
|
| - Node* base = HeapConstant(array);
|
| - Node* vector = UndefinedConstant();
|
| - Node* context = UndefinedConstant();
|
| - Node* effect = graph()->start();
|
| - Node* control = graph()->start();
|
| - Reduction r = Reduce(
|
| - graph()->NewNode(javascript()->LoadProperty(language_mode, feedback),
|
| - base, key, vector, context, EmptyFrameState(),
|
| - EmptyFrameState(), effect, control));
|
| + Handle<JSTypedArray> array =
|
| + factory()->NewJSTypedArray(type, buffer, 0, kLength);
|
| + int const element_size = static_cast<int>(array->element_size());
|
| +
|
| + Node* key = Parameter(
|
| + Type::Range(kMinInt / element_size, kMaxInt / element_size, zone()));
|
| + Node* base = HeapConstant(array);
|
| + Node* vector = UndefinedConstant();
|
| + Node* context = UndefinedConstant();
|
| + Node* effect = graph()->start();
|
| + Node* control = graph()->start();
|
| + Reduction r = Reduce(graph()->NewNode(
|
| + javascript()->LoadProperty(feedback), base, key, vector, context,
|
| + EmptyFrameState(), EmptyFrameState(), effect, control));
|
|
|
| - Matcher<Node*> offset_matcher =
|
| - element_size == 1
|
| - ? key
|
| - : IsWord32Shl(key, IsInt32Constant(WhichPowerOf2(element_size)));
|
| + Matcher<Node*> offset_matcher =
|
| + element_size == 1
|
| + ? key
|
| + : IsWord32Shl(key, IsInt32Constant(WhichPowerOf2(element_size)));
|
|
|
| - ASSERT_TRUE(r.Changed());
|
| - EXPECT_THAT(
|
| - r.replacement(),
|
| - IsLoadBuffer(BufferAccess(type),
|
| - IsIntPtrConstant(bit_cast<intptr_t>(&backing_store[0])),
|
| - offset_matcher,
|
| - IsNumberConstant(array->byte_length()->Number()), effect,
|
| - control));
|
| - }
|
| + ASSERT_TRUE(r.Changed());
|
| + EXPECT_THAT(
|
| + r.replacement(),
|
| + IsLoadBuffer(BufferAccess(type),
|
| + IsIntPtrConstant(bit_cast<intptr_t>(&backing_store[0])),
|
| + offset_matcher,
|
| + IsNumberConstant(array->byte_length()->Number()), effect,
|
| + control));
|
| }
|
| }
|
|
|
| @@ -642,32 +639,29 @@ TEST_F(JSTypedLoweringTest, JSLoadPropertyFromExternalTypedArrayWithSafeKey) {
|
| NewArrayBuffer(backing_store, sizeof(backing_store));
|
| VectorSlotPair feedback;
|
| TRACED_FOREACH(ExternalArrayType, type, kExternalArrayTypes) {
|
| - TRACED_FOREACH(LanguageMode, language_mode, kLanguageModes) {
|
| - Handle<JSTypedArray> array =
|
| - factory()->NewJSTypedArray(type, buffer, 0, kLength);
|
| - ElementAccess access = AccessBuilder::ForTypedArrayElement(type, true);
|
| -
|
| - int min = random_number_generator()->NextInt(static_cast<int>(kLength));
|
| - int max = random_number_generator()->NextInt(static_cast<int>(kLength));
|
| - if (min > max) std::swap(min, max);
|
| - Node* key = Parameter(Type::Range(min, max, zone()));
|
| - Node* base = HeapConstant(array);
|
| - Node* vector = UndefinedConstant();
|
| - Node* context = UndefinedConstant();
|
| - Node* effect = graph()->start();
|
| - Node* control = graph()->start();
|
| - Reduction r = Reduce(
|
| - graph()->NewNode(javascript()->LoadProperty(language_mode, feedback),
|
| - base, key, vector, context, EmptyFrameState(),
|
| - EmptyFrameState(), effect, control));
|
| + Handle<JSTypedArray> array =
|
| + factory()->NewJSTypedArray(type, buffer, 0, kLength);
|
| + ElementAccess access = AccessBuilder::ForTypedArrayElement(type, true);
|
| +
|
| + int min = random_number_generator()->NextInt(static_cast<int>(kLength));
|
| + int max = random_number_generator()->NextInt(static_cast<int>(kLength));
|
| + if (min > max) std::swap(min, max);
|
| + Node* key = Parameter(Type::Range(min, max, zone()));
|
| + Node* base = HeapConstant(array);
|
| + Node* vector = UndefinedConstant();
|
| + Node* context = UndefinedConstant();
|
| + Node* effect = graph()->start();
|
| + Node* control = graph()->start();
|
| + Reduction r = Reduce(graph()->NewNode(
|
| + javascript()->LoadProperty(feedback), base, key, vector, context,
|
| + EmptyFrameState(), EmptyFrameState(), effect, control));
|
|
|
| - ASSERT_TRUE(r.Changed());
|
| - EXPECT_THAT(
|
| - r.replacement(),
|
| - IsLoadElement(access,
|
| - IsIntPtrConstant(bit_cast<intptr_t>(&backing_store[0])),
|
| - key, effect, control));
|
| - }
|
| + ASSERT_TRUE(r.Changed());
|
| + EXPECT_THAT(
|
| + r.replacement(),
|
| + IsLoadElement(access,
|
| + IsIntPtrConstant(bit_cast<intptr_t>(&backing_store[0])),
|
| + key, effect, control));
|
| }
|
| }
|
|
|
| @@ -820,15 +814,12 @@ TEST_F(JSTypedLoweringTest, JSLoadNamedStringLength) {
|
| Node* const context = UndefinedConstant();
|
| Node* const effect = graph()->start();
|
| Node* const control = graph()->start();
|
| - TRACED_FOREACH(LanguageMode, language_mode, kLanguageModes) {
|
| - Reduction const r = Reduce(
|
| - graph()->NewNode(javascript()->LoadNamed(language_mode, name, feedback),
|
| - receiver, vector, context, EmptyFrameState(),
|
| - EmptyFrameState(), effect, control));
|
| - ASSERT_TRUE(r.Changed());
|
| - EXPECT_THAT(r.replacement(), IsLoadField(AccessBuilder::ForStringLength(),
|
| - receiver, effect, control));
|
| - }
|
| + Reduction const r = Reduce(graph()->NewNode(
|
| + javascript()->LoadNamed(name, feedback), receiver, vector, context,
|
| + EmptyFrameState(), EmptyFrameState(), effect, control));
|
| + ASSERT_TRUE(r.Changed());
|
| + EXPECT_THAT(r.replacement(), IsLoadField(AccessBuilder::ForStringLength(),
|
| + receiver, effect, control));
|
| }
|
|
|
|
|
| @@ -842,14 +833,11 @@ TEST_F(JSTypedLoweringTest, JSLoadNamedFunctionPrototype) {
|
| Node* const context = Parameter(Type::Internal(), 2);
|
| Node* const effect = graph()->start();
|
| Node* const control = graph()->start();
|
| - TRACED_FOREACH(LanguageMode, language_mode, kLanguageModes) {
|
| - Reduction const r = Reduce(
|
| - graph()->NewNode(javascript()->LoadNamed(language_mode, name, feedback),
|
| - receiver, vector, context, EmptyFrameState(),
|
| - EmptyFrameState(), effect, control));
|
| - ASSERT_TRUE(r.Changed());
|
| - EXPECT_THAT(r.replacement(), IsHeapConstant(function_prototype));
|
| - }
|
| + Reduction const r = Reduce(graph()->NewNode(
|
| + javascript()->LoadNamed(name, feedback), receiver, vector, context,
|
| + EmptyFrameState(), EmptyFrameState(), effect, control));
|
| + ASSERT_TRUE(r.Changed());
|
| + EXPECT_THAT(r.replacement(), IsHeapConstant(function_prototype));
|
| }
|
|
|
|
|
|
|