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

Unified Diff: test/cctest/compiler/test-loop-analysis.cc

Issue 2473643002: Revert of [turbofan] Support variable size argument popping in TF-generated functions (Closed)
Patch Set: Created 4 years, 1 month 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 | « test/cctest/compiler/test-js-typed-lowering.cc ('k') | test/cctest/compiler/test-representation-change.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/cctest/compiler/test-loop-analysis.cc
diff --git a/test/cctest/compiler/test-loop-analysis.cc b/test/cctest/compiler/test-loop-analysis.cc
index ffb08722698878001c0de6fad7e7ee4e0b94473f..8a845217df90c2e1d4c43c726269dbecdb31a5dc 100644
--- a/test/cctest/compiler/test-loop-analysis.cc
+++ b/test/cctest/compiler/test-loop-analysis.cc
@@ -116,8 +116,7 @@
}
Node* Return(Node* val, Node* effect, Node* control) {
- Node* zero = graph.NewNode(common.Int32Constant(0));
- Node* ret = graph.NewNode(common.Return(), zero, val, effect, control);
+ Node* ret = graph.NewNode(common.Return(), val, effect, control);
end->ReplaceInput(0, ret);
return ret;
}
@@ -697,8 +696,7 @@
Node* if_true = t.graph.NewNode(t.common.IfTrue(), branch);
Node* exit = t.graph.NewNode(t.common.IfFalse(), branch);
loop->ReplaceInput(1, if_true);
- Node* zero = t.graph.NewNode(t.common.Int32Constant(0));
- Node* ret = t.graph.NewNode(t.common.Return(), zero, p3, t.start, exit);
+ Node* ret = t.graph.NewNode(t.common.Return(), p3, t.start, exit);
t.graph.SetEnd(ret);
Node* choices[] = {p1, phi, cond};
@@ -745,9 +743,7 @@
loop2->ReplaceInput(1, if_true2);
loop1->ReplaceInput(1, exit2);
- Node* zero = t.graph.NewNode(t.common.Int32Constant(0));
- Node* ret =
- t.graph.NewNode(t.common.Return(), zero, phi1, t.start, exit1);
+ Node* ret = t.graph.NewNode(t.common.Return(), phi1, t.start, exit1);
t.graph.SetEnd(ret);
Node* choices[] = {p1, phi1, cond1, phi2, cond2};
@@ -834,8 +830,7 @@
loop2->ReplaceInput(1, exit3);
loop1->ReplaceInput(1, exit2);
- Node* zero = t.graph.NewNode(t.common.Int32Constant(0));
- Node* ret = t.graph.NewNode(t.common.Return(), zero, phi1, t.start, exit1);
+ Node* ret = t.graph.NewNode(t.common.Return(), phi1, t.start, exit1);
t.graph.SetEnd(ret);
// Mutate the graph according to the edge choices.
@@ -948,8 +943,7 @@
last = exit;
}
- Node* zero = t.graph.NewNode(t.common.Int32Constant(0));
- Node* ret = t.graph.NewNode(t.common.Return(), zero, t.p0, t.start, last);
+ Node* ret = t.graph.NewNode(t.common.Return(), t.p0, t.start, last);
t.graph.SetEnd(ret);
// Verify loops.
@@ -968,7 +962,6 @@
Node* entry = t.start;
// Build loops.
- Node* zero = t.graph.NewNode(t.common.Int32Constant(0));
for (int i = 0; i < count; i++) {
Node* loop = t.graph.NewNode(t.common.Loop(2), entry, t.start);
Node* phi = t.graph.NewNode(t.common.Phi(MachineRepresentation::kWord32, 2),
@@ -988,7 +981,7 @@
outer->ReplaceInput(1, exit);
} else {
// outer loop.
- Node* ret = t.graph.NewNode(t.common.Return(), zero, t.p0, t.start, exit);
+ Node* ret = t.graph.NewNode(t.common.Return(), t.p0, t.start, exit);
t.graph.SetEnd(ret);
}
outer = loop;
« no previous file with comments | « test/cctest/compiler/test-js-typed-lowering.cc ('k') | test/cctest/compiler/test-representation-change.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698