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

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

Issue 594443002: [turbofan] Disable value numbering for now. (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Created 6 years, 3 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 | no next file » | 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 "src/base/platform/elapsed-timer.h" 7 #include "src/base/platform/elapsed-timer.h"
8 #include "src/compiler/ast-graph-builder.h" 8 #include "src/compiler/ast-graph-builder.h"
9 #include "src/compiler/change-lowering.h" 9 #include "src/compiler/change-lowering.h"
10 #include "src/compiler/code-generator.h" 10 #include "src/compiler/code-generator.h"
(...skipping 251 matching lines...) Expand 10 before | Expand all | Expand 10 after
262 262
263 VerifyAndPrintGraph(&graph, "Lowered simplified"); 263 VerifyAndPrintGraph(&graph, "Lowered simplified");
264 } 264 }
265 { 265 {
266 // Lower changes that have been inserted before. 266 // Lower changes that have been inserted before.
267 PhaseStats lowering_stats(info(), PhaseStats::OPTIMIZATION, 267 PhaseStats lowering_stats(info(), PhaseStats::OPTIMIZATION,
268 "change lowering"); 268 "change lowering");
269 SourcePositionTable::Scope pos(&source_positions, 269 SourcePositionTable::Scope pos(&source_positions,
270 SourcePosition::Unknown()); 270 SourcePosition::Unknown());
271 Linkage linkage(info()); 271 Linkage linkage(info());
272 ValueNumberingReducer vn_reducer(zone()); 272 // TODO(turbofan): Value numbering disabled for now.
273 // ValueNumberingReducer vn_reducer(zone());
273 SimplifiedOperatorReducer simple_reducer(&jsgraph); 274 SimplifiedOperatorReducer simple_reducer(&jsgraph);
274 ChangeLowering lowering(&jsgraph, &linkage); 275 ChangeLowering lowering(&jsgraph, &linkage);
275 MachineOperatorReducer mach_reducer(&jsgraph); 276 MachineOperatorReducer mach_reducer(&jsgraph);
276 GraphReducer graph_reducer(&graph); 277 GraphReducer graph_reducer(&graph);
277 // TODO(titzer): Figure out if we should run all reducers at once here. 278 // TODO(titzer): Figure out if we should run all reducers at once here.
278 graph_reducer.AddReducer(&vn_reducer); 279 // graph_reducer.AddReducer(&vn_reducer);
279 graph_reducer.AddReducer(&simple_reducer); 280 graph_reducer.AddReducer(&simple_reducer);
280 graph_reducer.AddReducer(&lowering); 281 graph_reducer.AddReducer(&lowering);
281 graph_reducer.AddReducer(&mach_reducer); 282 graph_reducer.AddReducer(&mach_reducer);
282 graph_reducer.ReduceGraph(); 283 graph_reducer.ReduceGraph();
283 284
284 VerifyAndPrintGraph(&graph, "Lowered changes"); 285 VerifyAndPrintGraph(&graph, "Lowered changes");
285 } 286 }
286 } 287 }
287 288
288 Handle<Code> code = Handle<Code>::null(); 289 Handle<Code> code = Handle<Code>::null();
(...skipping 123 matching lines...) Expand 10 before | Expand all | Expand 10 after
412 } 413 }
413 414
414 415
415 void Pipeline::TearDown() { 416 void Pipeline::TearDown() {
416 InstructionOperand::TearDownCaches(); 417 InstructionOperand::TearDownCaches();
417 } 418 }
418 419
419 } // namespace compiler 420 } // namespace compiler
420 } // namespace internal 421 } // namespace internal
421 } // namespace v8 422 } // namespace v8
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698