Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(338)

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: test/cctest/compiler/test-js-typed-lowering.cc

Issue 1144183004: [strong] Refactor ObjectStrength into a replacement for strong boolean args (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: rebase Created 5 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« src/compiler/js-operator.h ('K') | « test/cctest/compiler/test-js-context-specialization.cc ('k') | test/cctest/test-heap.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/cctest/compiler/test-js-typed-lowering.cc
diff --git a/test/cctest/compiler/test-js-typed-lowering.cc b/test/cctest/compiler/test-js-typed-lowering.cc
index 058c0ee903602857cdf256fedd954b1ddd0c4d5d..a983cb80899261fc6786a74b1fec5f08243b753c 100644
--- a/test/cctest/compiler/test-js-typed-lowering.cc
+++ b/test/cctest/compiler/test-js-typed-lowering.cc
@@ -14,16 +14,16 @@
using namespace v8::internal;
using namespace v8::internal::compiler;
-#ifndef TEST_WITH_STRONG
-#define TEST_WITH_STRONG(Name) \
- static void Test##Name(); \
- static void TestWithStrong##Name(LanguageMode language_mode); \
- CcTest register_test_##Name(Test##Name, __FILE__, #Name, NULL, true, true); \
- static void Test##Name() { \
- TestWithStrong##Name(LanguageMode::SLOPPY); \
- TestWithStrong##Name(LanguageMode::STRONG); \
- } \
- static void TestWithStrong##Name(LanguageMode language_mode)
+#ifndef TEST_WITH_STRENGTH
+#define TEST_WITH_STRENGTH(Name) \
+ static void Test##Name(); \
+ static void TestWithStrength##Name(Strength strength); \
+ CcTest register_test_##Name(Test##Name, __FILE__, #Name, NULL, true, true); \
+ static void Test##Name() { \
+ TestWithStrength##Name(Strength::NORMAL); \
+ TestWithStrength##Name(Strength::STRONG); \
+ } \
+ static void TestWithStrength##Name(Strength strength)
#endif
@@ -233,7 +233,7 @@ static IrOpcode::Value NumberToI32(bool is_signed) {
// TODO(turbofan): Lowering of StringAdd is disabled for now.
#if 0
-TEST_WITH_STRONG(StringBinops) {
+TEST_WITH_STRENGTH(StringBinops) {
JSTypedLoweringTester R;
for (size_t i = 0; i < arraysize(kStringTypes); ++i) {
@@ -242,7 +242,7 @@ TEST_WITH_STRONG(StringBinops) {
for (size_t j = 0; j < arraysize(kStringTypes); ++j) {
Node* p1 = R.Parameter(kStringTypes[j], 1);
- Node* add = R.Binop(R.javascript.Add(language_mode), p0, p1);
+ Node* add = R.Binop(R.javascript.Add(strength), p0, p1);
Node* r = R.reduce(add);
R.CheckPureBinop(IrOpcode::kStringAdd, r);
@@ -254,12 +254,12 @@ TEST_WITH_STRONG(StringBinops) {
#endif
-TEST_WITH_STRONG(AddNumber1) {
+TEST_WITH_STRENGTH(AddNumber1) {
JSTypedLoweringTester R;
for (size_t i = 0; i < arraysize(kNumberTypes); ++i) {
Node* p0 = R.Parameter(kNumberTypes[i], 0);
Node* p1 = R.Parameter(kNumberTypes[i], 1);
- Node* add = R.Binop(R.javascript.Add(language_mode), p0, p1);
+ Node* add = R.Binop(R.javascript.Add(strength), p0, p1);
Node* r = R.reduce(add);
R.CheckPureBinop(IrOpcode::kNumberAdd, r);
@@ -269,14 +269,14 @@ TEST_WITH_STRONG(AddNumber1) {
}
-TEST_WITH_STRONG(NumberBinops) {
+TEST_WITH_STRENGTH(NumberBinops) {
JSTypedLoweringTester R;
const Operator* ops[] = {
- R.javascript.Add(language_mode), R.simplified.NumberAdd(),
- R.javascript.Subtract(language_mode), R.simplified.NumberSubtract(),
- R.javascript.Multiply(language_mode), R.simplified.NumberMultiply(),
- R.javascript.Divide(language_mode), R.simplified.NumberDivide(),
- R.javascript.Modulus(language_mode), R.simplified.NumberModulus(),
+ R.javascript.Add(strength), R.simplified.NumberAdd(),
+ R.javascript.Subtract(strength), R.simplified.NumberSubtract(),
+ R.javascript.Multiply(strength), R.simplified.NumberMultiply(),
+ R.javascript.Divide(strength), R.simplified.NumberDivide(),
+ R.javascript.Modulus(strength), R.simplified.NumberModulus(),
};
for (size_t i = 0; i < arraysize(kNumberTypes); ++i) {
@@ -316,14 +316,14 @@ static void CheckToI32(Node* old_input, Node* new_input, bool is_signed) {
// A helper class for testing lowering of bitwise shift operators.
class JSBitwiseShiftTypedLoweringTester : public JSTypedLoweringTester {
public:
- explicit JSBitwiseShiftTypedLoweringTester(LanguageMode language_mode)
- : JSTypedLoweringTester(), language_mode_(language_mode) {
+ explicit JSBitwiseShiftTypedLoweringTester(Strength strength)
+ : JSTypedLoweringTester(), strength_(strength) {
int i = 0;
- set(i++, javascript.ShiftLeft(language_mode_), true);
+ set(i++, javascript.ShiftLeft(strength_), true);
set(i++, machine.Word32Shl(), false);
- set(i++, javascript.ShiftRight(language_mode_), true);
+ set(i++, javascript.ShiftRight(strength_), true);
set(i++, machine.Word32Sar(), false);
- set(i++, javascript.ShiftRightLogical(language_mode_), false);
+ set(i++, javascript.ShiftRightLogical(strength_), false);
set(i++, machine.Word32Shr(), false);
}
static const int kNumberOps = 6;
@@ -331,7 +331,7 @@ class JSBitwiseShiftTypedLoweringTester : public JSTypedLoweringTester {
bool signedness[kNumberOps];
private:
- LanguageMode language_mode_;
+ Strength strength_;
void set(int idx, const Operator* op, bool s) {
ops[idx] = op;
signedness[idx] = s;
@@ -340,7 +340,7 @@ class JSBitwiseShiftTypedLoweringTester : public JSTypedLoweringTester {
TEST(Int32BitwiseShifts) {
- JSBitwiseShiftTypedLoweringTester R(LanguageMode::SLOPPY);
+ JSBitwiseShiftTypedLoweringTester R(Strength::NORMAL);
Type* types[] = {
Type::SignedSmall(), Type::UnsignedSmall(), Type::Negative32(),
@@ -381,14 +381,14 @@ TEST(Int32BitwiseShifts) {
// A helper class for testing lowering of bitwise operators.
class JSBitwiseTypedLoweringTester : public JSTypedLoweringTester {
public:
- explicit JSBitwiseTypedLoweringTester(LanguageMode language_mode)
- : JSTypedLoweringTester(), language_mode_(language_mode) {
+ explicit JSBitwiseTypedLoweringTester(Strength strength)
+ : JSTypedLoweringTester(), strength_(strength) {
int i = 0;
- set(i++, javascript.BitwiseOr(language_mode_), true);
+ set(i++, javascript.BitwiseOr(strength_), true);
set(i++, machine.Word32Or(), true);
- set(i++, javascript.BitwiseXor(language_mode_), true);
+ set(i++, javascript.BitwiseXor(strength_), true);
set(i++, machine.Word32Xor(), true);
- set(i++, javascript.BitwiseAnd(language_mode_), true);
+ set(i++, javascript.BitwiseAnd(strength_), true);
set(i++, machine.Word32And(), true);
}
static const int kNumberOps = 6;
@@ -396,7 +396,7 @@ class JSBitwiseTypedLoweringTester : public JSTypedLoweringTester {
bool signedness[kNumberOps];
private:
- LanguageMode language_mode_;
+ Strength strength_;
void set(int idx, const Operator* op, bool s) {
ops[idx] = op;
signedness[idx] = s;
@@ -405,7 +405,7 @@ class JSBitwiseTypedLoweringTester : public JSTypedLoweringTester {
TEST(Int32BitwiseBinops) {
- JSBitwiseTypedLoweringTester R(LanguageMode::SLOPPY);
+ JSBitwiseTypedLoweringTester R(Strength::NORMAL);
Type* types[] = {
Type::SignedSmall(), Type::UnsignedSmall(), Type::Unsigned32(),
@@ -597,15 +597,14 @@ TEST(JSToString_replacement) {
}
-TEST_WITH_STRONG(StringComparison) {
+TEST_WITH_STRENGTH(StringComparison) {
JSTypedLoweringTester R;
const Operator* ops[] = {
- R.javascript.LessThan(language_mode), R.simplified.StringLessThan(),
- R.javascript.LessThanOrEqual(language_mode),
- R.simplified.StringLessThanOrEqual(),
- R.javascript.GreaterThan(language_mode), R.simplified.StringLessThan(),
- R.javascript.GreaterThanOrEqual(language_mode),
+ R.javascript.LessThan(strength), R.simplified.StringLessThan(),
+ R.javascript.LessThanOrEqual(strength),
+ R.simplified.StringLessThanOrEqual(), R.javascript.GreaterThan(strength),
+ R.simplified.StringLessThan(), R.javascript.GreaterThanOrEqual(strength),
R.simplified.StringLessThanOrEqual()};
for (size_t i = 0; i < arraysize(kStringTypes); i++) {
@@ -647,15 +646,14 @@ static void CheckIsConvertedToNumber(Node* val, Node* converted) {
}
-TEST_WITH_STRONG(NumberComparison) {
+TEST_WITH_STRENGTH(NumberComparison) {
JSTypedLoweringTester R;
const Operator* ops[] = {
- R.javascript.LessThan(language_mode), R.simplified.NumberLessThan(),
- R.javascript.LessThanOrEqual(language_mode),
- R.simplified.NumberLessThanOrEqual(),
- R.javascript.GreaterThan(language_mode), R.simplified.NumberLessThan(),
- R.javascript.GreaterThanOrEqual(language_mode),
+ R.javascript.LessThan(strength), R.simplified.NumberLessThan(),
+ R.javascript.LessThanOrEqual(strength),
+ R.simplified.NumberLessThanOrEqual(), R.javascript.GreaterThan(strength),
+ R.simplified.NumberLessThan(), R.javascript.GreaterThanOrEqual(strength),
R.simplified.NumberLessThanOrEqual()};
Node* const p0 = R.Parameter(Type::Number(), 0);
@@ -679,7 +677,7 @@ TEST_WITH_STRONG(NumberComparison) {
}
-TEST_WITH_STRONG(MixedComparison1) {
+TEST_WITH_STRENGTH(MixedComparison1) {
JSTypedLoweringTester R;
Type* types[] = {Type::Number(), Type::String(),
@@ -691,14 +689,14 @@ TEST_WITH_STRONG(MixedComparison1) {
for (size_t j = 0; j < arraysize(types); j++) {
Node* p1 = R.Parameter(types[j], 1);
{
- const Operator* less_than = R.javascript.LessThan(language_mode);
+ const Operator* less_than = R.javascript.LessThan(strength);
Node* cmp = R.Binop(less_than, p0, p1);
Node* r = R.reduce(cmp);
if (types[i]->Is(Type::String()) && types[j]->Is(Type::String())) {
R.CheckPureBinop(R.simplified.StringLessThan(), r);
} else if ((types[i]->Is(Type::Number()) &&
types[j]->Is(Type::Number())) ||
- (!is_strong(language_mode) &&
+ (!is_strong(strength) &&
(!types[i]->Maybe(Type::String()) ||
!types[j]->Maybe(Type::String())))) {
R.CheckPureBinop(R.simplified.NumberLessThan(), r);
@@ -712,7 +710,7 @@ TEST_WITH_STRONG(MixedComparison1) {
}
-TEST_WITH_STRONG(RemoveToNumberEffects) {
+TEST_WITH_STRENGTH(RemoveToNumberEffects) {
JSTypedLoweringTester R;
Node* effect_use = NULL;
@@ -738,14 +736,14 @@ TEST_WITH_STRONG(RemoveToNumberEffects) {
case 2:
effect_use = R.graph.NewNode(R.common.EffectPhi(1), ton, R.start());
case 3:
- effect_use = R.graph.NewNode(R.javascript.Add(language_mode), ton, ton,
- R.context(), frame_state, frame_state, ton,
- R.start());
+ effect_use =
+ R.graph.NewNode(R.javascript.Add(strength), ton, ton, R.context(),
+ frame_state, frame_state, ton, R.start());
break;
case 4:
- effect_use = R.graph.NewNode(R.javascript.Add(language_mode), p0, p0,
- R.context(), frame_state, frame_state, ton,
- R.start());
+ effect_use =
+ R.graph.NewNode(R.javascript.Add(strength), p0, p0, R.context(),
+ frame_state, frame_state, ton, R.start());
break;
case 5:
effect_use = R.graph.NewNode(R.common.Return(), p0, ton, R.start());
@@ -903,18 +901,18 @@ TEST(StringEquality) {
}
-TEST_WITH_STRONG(RemovePureNumberBinopEffects) {
+TEST_WITH_STRENGTH(RemovePureNumberBinopEffects) {
JSTypedLoweringTester R;
const Operator* ops[] = {
- R.javascript.Equal(), R.simplified.NumberEqual(),
- R.javascript.Add(language_mode), R.simplified.NumberAdd(),
- R.javascript.Subtract(language_mode), R.simplified.NumberSubtract(),
- R.javascript.Multiply(language_mode), R.simplified.NumberMultiply(),
- R.javascript.Divide(language_mode), R.simplified.NumberDivide(),
- R.javascript.Modulus(language_mode), R.simplified.NumberModulus(),
- R.javascript.LessThan(language_mode), R.simplified.NumberLessThan(),
- R.javascript.LessThanOrEqual(language_mode),
+ R.javascript.Equal(), R.simplified.NumberEqual(),
+ R.javascript.Add(strength), R.simplified.NumberAdd(),
+ R.javascript.Subtract(strength), R.simplified.NumberSubtract(),
+ R.javascript.Multiply(strength), R.simplified.NumberMultiply(),
+ R.javascript.Divide(strength), R.simplified.NumberDivide(),
+ R.javascript.Modulus(strength), R.simplified.NumberModulus(),
+ R.javascript.LessThan(strength), R.simplified.NumberLessThan(),
+ R.javascript.LessThanOrEqual(strength),
R.simplified.NumberLessThanOrEqual(),
};
@@ -936,13 +934,9 @@ TEST(OrderNumberBinopEffects1) {
JSTypedLoweringTester R;
const Operator* ops[] = {
- R.javascript.Subtract(LanguageMode::SLOPPY),
- R.simplified.NumberSubtract(),
- R.javascript.Multiply(LanguageMode::SLOPPY),
- R.simplified.NumberMultiply(),
- R.javascript.Divide(LanguageMode::SLOPPY),
- R.simplified.NumberDivide(),
- };
+ R.javascript.Subtract(Strength::NORMAL), R.simplified.NumberSubtract(),
+ R.javascript.Multiply(Strength::NORMAL), R.simplified.NumberMultiply(),
+ R.javascript.Divide(Strength::NORMAL), R.simplified.NumberDivide()};
for (size_t j = 0; j < arraysize(ops); j += 2) {
BinopEffectsTester B(ops[j], Type::Symbol(), Type::Symbol());
@@ -964,15 +958,10 @@ TEST(OrderNumberBinopEffects2) {
JSTypedLoweringTester R;
const Operator* ops[] = {
- R.javascript.Add(LanguageMode::SLOPPY),
- R.simplified.NumberAdd(),
- R.javascript.Subtract(LanguageMode::SLOPPY),
- R.simplified.NumberSubtract(),
- R.javascript.Multiply(LanguageMode::SLOPPY),
- R.simplified.NumberMultiply(),
- R.javascript.Divide(LanguageMode::SLOPPY),
- R.simplified.NumberDivide(),
- };
+ R.javascript.Add(Strength::NORMAL), R.simplified.NumberAdd(),
+ R.javascript.Subtract(Strength::NORMAL), R.simplified.NumberSubtract(),
+ R.javascript.Multiply(Strength::NORMAL), R.simplified.NumberMultiply(),
+ R.javascript.Divide(Strength::NORMAL), R.simplified.NumberDivide()};
for (size_t j = 0; j < arraysize(ops); j += 2) {
BinopEffectsTester B(ops[j], Type::Number(), Type::Symbol());
@@ -1006,9 +995,8 @@ TEST(OrderCompareEffects) {
JSTypedLoweringTester R;
const Operator* ops[] = {
- R.javascript.GreaterThan(LanguageMode::SLOPPY),
- R.simplified.NumberLessThan(),
- R.javascript.GreaterThanOrEqual(LanguageMode::SLOPPY),
+ R.javascript.GreaterThan(Strength::NORMAL), R.simplified.NumberLessThan(),
+ R.javascript.GreaterThanOrEqual(Strength::NORMAL),
R.simplified.NumberLessThanOrEqual(),
};
@@ -1056,7 +1044,7 @@ TEST(OrderCompareEffects) {
TEST(Int32BinopEffects) {
- JSBitwiseTypedLoweringTester R(LanguageMode::SLOPPY);
+ JSBitwiseTypedLoweringTester R(Strength::NORMAL);
for (int j = 0; j < R.kNumberOps; j += 2) {
bool signed_left = R.signedness[j], signed_right = R.signedness[j + 1];
BinopEffectsTester B(R.ops[j], I32Type(signed_left), I32Type(signed_right));
@@ -1137,9 +1125,9 @@ TEST(Int32BinopEffects) {
}
-TEST_WITH_STRONG(Int32AddNarrowing) {
+TEST_WITH_STRENGTH(Int32AddNarrowing) {
{
- JSBitwiseTypedLoweringTester R(language_mode);
+ JSBitwiseTypedLoweringTester R(strength);
for (int o = 0; o < R.kNumberOps; o += 2) {
for (size_t i = 0; i < arraysize(kInt32Types); i++) {
@@ -1162,7 +1150,7 @@ TEST_WITH_STRONG(Int32AddNarrowing) {
}
}
{
- JSBitwiseShiftTypedLoweringTester R(language_mode);
+ JSBitwiseShiftTypedLoweringTester R(strength);
for (int o = 0; o < R.kNumberOps; o += 2) {
for (size_t i = 0; i < arraysize(kInt32Types); i++) {
@@ -1185,7 +1173,7 @@ TEST_WITH_STRONG(Int32AddNarrowing) {
}
}
{
- JSBitwiseTypedLoweringTester R(language_mode);
+ JSBitwiseTypedLoweringTester R(strength);
for (int o = 0; o < R.kNumberOps; o += 2) {
Node* n0 = R.Parameter(I32Type(R.signedness[o]));
@@ -1209,7 +1197,7 @@ TEST_WITH_STRONG(Int32AddNarrowing) {
}
-TEST_WITH_STRONG(Int32Comparisons) {
+TEST_WITH_STRENGTH(Int32Comparisons) {
JSTypedLoweringTester R;
struct Entry {
@@ -1221,17 +1209,16 @@ TEST_WITH_STRONG(Int32Comparisons) {
};
Entry ops[] = {
- {R.javascript.LessThan(language_mode), R.machine.Uint32LessThan(),
+ {R.javascript.LessThan(strength), R.machine.Uint32LessThan(),
R.machine.Int32LessThan(), R.simplified.NumberLessThan(), false},
- {R.javascript.LessThanOrEqual(language_mode),
+ {R.javascript.LessThanOrEqual(strength),
R.machine.Uint32LessThanOrEqual(), R.machine.Int32LessThanOrEqual(),
R.simplified.NumberLessThanOrEqual(), false},
- {R.javascript.GreaterThan(language_mode), R.machine.Uint32LessThan(),
+ {R.javascript.GreaterThan(strength), R.machine.Uint32LessThan(),
R.machine.Int32LessThan(), R.simplified.NumberLessThan(), true},
- {R.javascript.GreaterThanOrEqual(language_mode),
+ {R.javascript.GreaterThanOrEqual(strength),
R.machine.Uint32LessThanOrEqual(), R.machine.Int32LessThanOrEqual(),
- R.simplified.NumberLessThanOrEqual(), true}
- };
+ R.simplified.NumberLessThanOrEqual(), true}};
for (size_t o = 0; o < arraysize(ops); o++) {
for (size_t i = 0; i < arraysize(kNumberTypes); i++) {
« src/compiler/js-operator.h ('K') | « test/cctest/compiler/test-js-context-specialization.cc ('k') | test/cctest/test-heap.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698