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

Unified Diff: src/hydrogen.cc

Issue 21055011: First implementation of allocation elimination in Hydrogen. (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Minor cleanup. Created 7 years, 5 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 side-by-side diff with in-line comments
Download patch
Index: src/hydrogen.cc
diff --git a/src/hydrogen.cc b/src/hydrogen.cc
index f1eaa22c879d2093d0b67f58204c207a43ee5b02..4208727b9c390a4c6d30d2cc77efa6aead48c1bc 100644
--- a/src/hydrogen.cc
+++ b/src/hydrogen.cc
@@ -2949,6 +2949,7 @@ bool HGraph::Optimize(SmartArrayPointer<char>* bailout_reason) {
}
Run<HPropagateDeoptimizingMarkPhase>();
+
titzer 2013/08/01 17:11:50 whitespace change
Michael Starzinger 2013/08/05 15:13:00 Done. Removed. Even though I think there should be
if (!CheckConstPhiUses()) {
*bailout_reason = SmartArrayPointer<char>(StrDup(
"Unsupported phi use of const variable"));
@@ -2961,6 +2962,8 @@ bool HGraph::Optimize(SmartArrayPointer<char>* bailout_reason) {
return false;
}
+ if (FLAG_use_escape_analysis) Run<HEscapeAnalysisPhase>();
+
// Remove dead code and phis
if (FLAG_dead_code_elimination) Run<HDeadCodeEliminationPhase>();
CollectPhis();
@@ -2986,8 +2989,6 @@ bool HGraph::Optimize(SmartArrayPointer<char>* bailout_reason) {
if (FLAG_use_canonicalizing) Run<HCanonicalizePhase>();
- if (FLAG_use_escape_analysis) Run<HEscapeAnalysisPhase>();
-
if (FLAG_use_gvn) Run<HGlobalValueNumberingPhase>();
if (FLAG_use_range) Run<HRangeAnalysisPhase>();
« no previous file with comments | « src/heap.cc ('k') | src/hydrogen-escape-analysis.h » ('j') | src/hydrogen-escape-analysis.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698