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

Side by Side Diff: src/compiler/pipeline.cc

Issue 630423002: [turbofan] Reenable value numbering. (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Win fix. Created 6 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « no previous file | src/compiler/value-numbering-reducer.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2014 the V8 project authors. All rights reserved. 1 // Copyright 2014 the V8 project authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "src/compiler/pipeline.h" 5 #include "src/compiler/pipeline.h"
6 6
7 #include <sstream> 7 #include <sstream>
8 8
9 #include "src/base/platform/elapsed-timer.h" 9 #include "src/base/platform/elapsed-timer.h"
10 #include "src/compiler/ast-graph-builder.h" 10 #include "src/compiler/ast-graph-builder.h"
(...skipping 272 matching lines...) Expand 10 before | Expand all | Expand 10 after
283 283
284 VerifyAndPrintGraph(&graph, "Lowered simplified"); 284 VerifyAndPrintGraph(&graph, "Lowered simplified");
285 } 285 }
286 { 286 {
287 // Lower changes that have been inserted before. 287 // Lower changes that have been inserted before.
288 PhaseStats lowering_stats(info(), PhaseStats::OPTIMIZATION, 288 PhaseStats lowering_stats(info(), PhaseStats::OPTIMIZATION,
289 "change lowering"); 289 "change lowering");
290 SourcePositionTable::Scope pos(&source_positions, 290 SourcePositionTable::Scope pos(&source_positions,
291 SourcePosition::Unknown()); 291 SourcePosition::Unknown());
292 Linkage linkage(info()); 292 Linkage linkage(info());
293 // TODO(turbofan): Value numbering disabled for now. 293 ValueNumberingReducer vn_reducer(zone());
294 // ValueNumberingReducer vn_reducer(zone());
295 SimplifiedOperatorReducer simple_reducer(&jsgraph); 294 SimplifiedOperatorReducer simple_reducer(&jsgraph);
296 ChangeLowering lowering(&jsgraph, &linkage); 295 ChangeLowering lowering(&jsgraph, &linkage);
297 MachineOperatorReducer mach_reducer(&jsgraph); 296 MachineOperatorReducer mach_reducer(&jsgraph);
298 GraphReducer graph_reducer(&graph); 297 GraphReducer graph_reducer(&graph);
299 // TODO(titzer): Figure out if we should run all reducers at once here. 298 // TODO(titzer): Figure out if we should run all reducers at once here.
300 // graph_reducer.AddReducer(&vn_reducer); 299 graph_reducer.AddReducer(&vn_reducer);
301 graph_reducer.AddReducer(&simple_reducer); 300 graph_reducer.AddReducer(&simple_reducer);
302 graph_reducer.AddReducer(&lowering); 301 graph_reducer.AddReducer(&lowering);
303 graph_reducer.AddReducer(&mach_reducer); 302 graph_reducer.AddReducer(&mach_reducer);
304 graph_reducer.ReduceGraph(); 303 graph_reducer.ReduceGraph();
305 304
306 VerifyAndPrintGraph(&graph, "Lowered changes"); 305 VerifyAndPrintGraph(&graph, "Lowered changes");
307 } 306 }
308 } 307 }
309 308
310 { 309 {
(...skipping 136 matching lines...) Expand 10 before | Expand all | Expand 10 after
447 } 446 }
448 447
449 448
450 void Pipeline::TearDown() { 449 void Pipeline::TearDown() {
451 InstructionOperand::TearDownCaches(); 450 InstructionOperand::TearDownCaches();
452 } 451 }
453 452
454 } // namespace compiler 453 } // namespace compiler
455 } // namespace internal 454 } // namespace internal
456 } // namespace v8 455 } // namespace v8
OLDNEW
« no previous file with comments | « no previous file | src/compiler/value-numbering-reducer.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698