[turbofan] Also run MachineOperatorReducer in SimplifiedLoweringPhase.
authorBenedikt Meurer <bmeurer@chromium.org>
Thu, 18 Dec 2014 07:47:22 +0000 (08:47 +0100)
committerBenedikt Meurer <bmeurer@chromium.org>
Thu, 18 Dec 2014 07:47:43 +0000 (07:47 +0000)
R=svenpanne@chromium.org

Review URL: https://codereview.chromium.org/809313002

Cr-Commit-Position: refs/heads/master@{#25874}

src/compiler/pipeline.cc

index 97ede38..165a737 100644 (file)
@@ -442,9 +442,11 @@ struct SimplifiedLoweringPhase {
     lowering.LowerAllNodes();
     ValueNumberingReducer vn_reducer(temp_zone);
     SimplifiedOperatorReducer simple_reducer(data->jsgraph());
+    MachineOperatorReducer machine_reducer(data->jsgraph());
     GraphReducer graph_reducer(data->graph(), temp_zone);
     graph_reducer.AddReducer(&vn_reducer);
     graph_reducer.AddReducer(&simple_reducer);
+    graph_reducer.AddReducer(&machine_reducer);
     graph_reducer.ReduceGraph();
   }
 };
@@ -460,13 +462,12 @@ struct ChangeLoweringPhase {
     ValueNumberingReducer vn_reducer(temp_zone);
     SimplifiedOperatorReducer simple_reducer(data->jsgraph());
     ChangeLowering lowering(data->jsgraph(), &linkage);
-    MachineOperatorReducer mach_reducer(data->jsgraph());
+    MachineOperatorReducer machine_reducer(data->jsgraph());
     GraphReducer graph_reducer(data->graph(), temp_zone);
-    // TODO(titzer): Figure out if we should run all reducers at once here.
     graph_reducer.AddReducer(&vn_reducer);
     graph_reducer.AddReducer(&simple_reducer);
     graph_reducer.AddReducer(&lowering);
-    graph_reducer.AddReducer(&mach_reducer);
+    graph_reducer.AddReducer(&machine_reducer);
     graph_reducer.ReduceGraph();
   }
 };