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

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

Issue 2342223003: Merged: [turbofan] Disable LoadElimination completely for asm.js. (Closed)
Patch Set: Created 4 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
« no previous file with comments | « no previous file | test/mjsunit/asm/load-elimination.js » ('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 <fstream> // NOLINT(readability/streams) 7 #include <fstream> // NOLINT(readability/streams)
8 #include <memory> 8 #include <memory>
9 #include <sstream> 9 #include <sstream>
10 10
(...skipping 1516 matching lines...) Expand 10 before | Expand all | Expand 10 after
1527 if (FLAG_turbo_stress_loop_peeling) { 1527 if (FLAG_turbo_stress_loop_peeling) {
1528 Run<StressLoopPeelingPhase>(); 1528 Run<StressLoopPeelingPhase>();
1529 RunPrintAndVerify("Loop peeled"); 1529 RunPrintAndVerify("Loop peeled");
1530 } 1530 }
1531 1531
1532 if (FLAG_turbo_escape) { 1532 if (FLAG_turbo_escape) {
1533 Run<EscapeAnalysisPhase>(); 1533 Run<EscapeAnalysisPhase>();
1534 RunPrintAndVerify("Escape Analysed"); 1534 RunPrintAndVerify("Escape Analysed");
1535 } 1535 }
1536 1536
1537 if (FLAG_turbo_load_elimination) { 1537 if (!info()->shared_info()->asm_function() && FLAG_turbo_load_elimination) {
1538 Run<LoadEliminationPhase>(); 1538 Run<LoadEliminationPhase>();
1539 RunPrintAndVerify("Load eliminated"); 1539 RunPrintAndVerify("Load eliminated");
1540 } 1540 }
1541 } 1541 }
1542 1542
1543 // Select representations. This has to run w/o the Typer decorator, because 1543 // Select representations. This has to run w/o the Typer decorator, because
1544 // we cannot compute meaningful types anyways, and the computed types might 1544 // we cannot compute meaningful types anyways, and the computed types might
1545 // even conflict with the representation/truncation logic. 1545 // even conflict with the representation/truncation logic.
1546 Run<RepresentationSelectionPhase>(); 1546 Run<RepresentationSelectionPhase>();
1547 RunPrintAndVerify("Representations selected", true); 1547 RunPrintAndVerify("Representations selected", true);
(...skipping 382 matching lines...) Expand 10 before | Expand all | Expand 10 after
1930 data->DeleteRegisterAllocationZone(); 1930 data->DeleteRegisterAllocationZone();
1931 } 1931 }
1932 1932
1933 CompilationInfo* PipelineImpl::info() const { return data_->info(); } 1933 CompilationInfo* PipelineImpl::info() const { return data_->info(); }
1934 1934
1935 Isolate* PipelineImpl::isolate() const { return info()->isolate(); } 1935 Isolate* PipelineImpl::isolate() const { return info()->isolate(); }
1936 1936
1937 } // namespace compiler 1937 } // namespace compiler
1938 } // namespace internal 1938 } // namespace internal
1939 } // namespace v8 1939 } // namespace v8
OLDNEW
« no previous file with comments | « no previous file | test/mjsunit/asm/load-elimination.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698