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

Unified Diff: test/cctest/compiler/test-run-machops.cc

Issue 2428443002: [wasm] Trim graph before scheduling. (Closed)
Patch Set: Use proper temp registers if Projection(1) does not exist. Created 4 years, 2 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
« no previous file with comments | « src/compiler/pipeline.cc ('k') | test/cctest/wasm/test-run-wasm.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/cctest/compiler/test-run-machops.cc
diff --git a/test/cctest/compiler/test-run-machops.cc b/test/cctest/compiler/test-run-machops.cc
index 50b46d7d0e288424e10ab97155590f346b6a4bf6..1beb1ad56c64cf1227368230bab2747fa2c73dec 100644
--- a/test/cctest/compiler/test-run-machops.cc
+++ b/test/cctest/compiler/test-run-machops.cc
@@ -4161,6 +4161,25 @@ TEST(RunInt32PairAdd) {
}
}
+TEST(RunInt32PairAddUseOnlyHighWord) {
+ BufferedRawMachineAssemblerTester<int32_t> m(
+ MachineType::Uint32(), MachineType::Uint32(), MachineType::Uint32(),
+ MachineType::Uint32());
+
+ m.Return(m.Projection(1, m.Int32PairAdd(m.Parameter(0), m.Parameter(1),
+ m.Parameter(2), m.Parameter(3))));
+
+ FOR_UINT64_INPUTS(i) {
+ FOR_UINT64_INPUTS(j) {
+ CHECK_EQ(static_cast<uint32_t>((*i + *j) >> 32),
+ m.Call(static_cast<uint32_t>(*i & 0xffffffff),
+ static_cast<uint32_t>(*i >> 32),
+ static_cast<uint32_t>(*j & 0xffffffff),
+ static_cast<uint32_t>(*j >> 32)));
+ }
+ }
+}
+
void TestInt32PairAddWithSharedInput(int a, int b, int c, int d) {
BufferedRawMachineAssemblerTester<int32_t> m(MachineType::Uint32(),
MachineType::Uint32());
@@ -4224,6 +4243,25 @@ TEST(RunInt32PairSub) {
}
}
+TEST(RunInt32PairSubUseOnlyHighWord) {
+ BufferedRawMachineAssemblerTester<int32_t> m(
+ MachineType::Uint32(), MachineType::Uint32(), MachineType::Uint32(),
+ MachineType::Uint32());
+
+ m.Return(m.Projection(1, m.Int32PairSub(m.Parameter(0), m.Parameter(1),
+ m.Parameter(2), m.Parameter(3))));
+
+ FOR_UINT64_INPUTS(i) {
+ FOR_UINT64_INPUTS(j) {
+ CHECK_EQ(static_cast<uint32_t>((*i - *j) >> 32),
+ m.Call(static_cast<uint32_t>(*i & 0xffffffff),
+ static_cast<uint32_t>(*i >> 32),
+ static_cast<uint32_t>(*j & 0xffffffff),
+ static_cast<uint32_t>(*j >> 32)));
+ }
+ }
+}
+
void TestInt32PairSubWithSharedInput(int a, int b, int c, int d) {
BufferedRawMachineAssemblerTester<int32_t> m(MachineType::Uint32(),
MachineType::Uint32());
@@ -4287,6 +4325,25 @@ TEST(RunInt32PairMul) {
}
}
+TEST(RunInt32PairMulUseOnlyHighWord) {
+ BufferedRawMachineAssemblerTester<int32_t> m(
+ MachineType::Uint32(), MachineType::Uint32(), MachineType::Uint32(),
+ MachineType::Uint32());
+
+ m.Return(m.Projection(1, m.Int32PairMul(m.Parameter(0), m.Parameter(1),
+ m.Parameter(2), m.Parameter(3))));
+
+ FOR_UINT64_INPUTS(i) {
+ FOR_UINT64_INPUTS(j) {
+ CHECK_EQ(static_cast<uint32_t>((*i * *j) >> 32),
+ m.Call(static_cast<uint32_t>(*i & 0xffffffff),
+ static_cast<uint32_t>(*i >> 32),
+ static_cast<uint32_t>(*j & 0xffffffff),
+ static_cast<uint32_t>(*j >> 32)));
+ }
+ }
+}
+
void TestInt32PairMulWithSharedInput(int a, int b, int c, int d) {
BufferedRawMachineAssemblerTester<int32_t> m(MachineType::Uint32(),
MachineType::Uint32());
@@ -4330,13 +4387,13 @@ TEST(RunWord32PairShl) {
uint32_t high;
uint32_t low;
- Node* PairAdd =
+ Node* PairShl =
m.Word32PairShl(m.Parameter(0), m.Parameter(1), m.Parameter(2));
m.StoreToPointer(&low, MachineRepresentation::kWord32,
- m.Projection(0, PairAdd));
+ m.Projection(0, PairShl));
m.StoreToPointer(&high, MachineRepresentation::kWord32,
- m.Projection(1, PairAdd));
+ m.Projection(1, PairShl));
m.Return(m.Int32Constant(74));
FOR_UINT64_INPUTS(i) {
@@ -4348,6 +4405,22 @@ TEST(RunWord32PairShl) {
}
}
+TEST(RunWord32PairShlUseOnlyHighWord) {
+ BufferedRawMachineAssemblerTester<int32_t> m(
+ MachineType::Uint32(), MachineType::Uint32(), MachineType::Uint32());
+
+ m.Return(m.Projection(
+ 1, m.Word32PairShl(m.Parameter(0), m.Parameter(1), m.Parameter(2))));
+
+ FOR_UINT64_INPUTS(i) {
+ for (uint32_t j = 0; j < 64; j++) {
+ CHECK_EQ(static_cast<uint32_t>((*i << j) >> 32),
+ m.Call(static_cast<uint32_t>(*i & 0xffffffff),
+ static_cast<uint32_t>(*i >> 32), j));
+ }
+ }
+}
+
void TestWord32PairShlWithSharedInput(int a, int b) {
BufferedRawMachineAssemblerTester<int32_t> m(MachineType::Uint32(),
MachineType::Uint32());
@@ -4405,6 +4478,22 @@ TEST(RunWord32PairShr) {
}
}
+TEST(RunWord32PairShrUseOnlyHighWord) {
+ BufferedRawMachineAssemblerTester<int32_t> m(
+ MachineType::Uint32(), MachineType::Uint32(), MachineType::Uint32());
+
+ m.Return(m.Projection(
+ 1, m.Word32PairShr(m.Parameter(0), m.Parameter(1), m.Parameter(2))));
+
+ FOR_UINT64_INPUTS(i) {
+ for (uint32_t j = 0; j < 64; j++) {
+ CHECK_EQ(static_cast<uint32_t>((*i >> j) >> 32),
+ m.Call(static_cast<uint32_t>(*i & 0xffffffff),
+ static_cast<uint32_t>(*i >> 32), j));
+ }
+ }
+}
+
TEST(RunWord32PairSar) {
BufferedRawMachineAssemblerTester<int32_t> m(
MachineType::Uint32(), MachineType::Uint32(), MachineType::Uint32());
@@ -4430,6 +4519,21 @@ TEST(RunWord32PairSar) {
}
}
+TEST(RunWord32PairSarUseOnlyHighWord) {
+ BufferedRawMachineAssemblerTester<int32_t> m(
+ MachineType::Uint32(), MachineType::Uint32(), MachineType::Uint32());
+
+ m.Return(m.Projection(
+ 1, m.Word32PairSar(m.Parameter(0), m.Parameter(1), m.Parameter(2))));
+
+ FOR_INT64_INPUTS(i) {
+ for (uint32_t j = 0; j < 64; j++) {
+ CHECK_EQ(static_cast<uint32_t>((*i >> j) >> 32),
+ m.Call(static_cast<uint32_t>(*i & 0xffffffff),
+ static_cast<uint32_t>(*i >> 32), j));
+ }
+ }
+}
#endif
TEST(RunDeadChangeFloat64ToInt32) {
« no previous file with comments | « src/compiler/pipeline.cc ('k') | test/cctest/wasm/test-run-wasm.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698