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

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: Ported to x64 and ARM architectures. Created 7 years, 4 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..a967084540287c6c5e8942d4c96bcb6b91403721 100644
--- a/src/hydrogen.cc
+++ b/src/hydrogen.cc
@@ -2961,6 +2961,8 @@ bool HGraph::Optimize(SmartArrayPointer<char>* bailout_reason) {
return false;
}
+ if (FLAG_use_escape_analysis) Run<HEscapeAnalysisPhase>();
titzer 2013/08/06 12:15:47 I think you might want to run after the first DCE
Michael Starzinger 2013/08/07 10:51:16 Done. Moved to after DCE.
+
// Remove dead code and phis
if (FLAG_dead_code_elimination) Run<HDeadCodeEliminationPhase>();
CollectPhis();
@@ -2986,8 +2988,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>();

Powered by Google App Engine
This is Rietveld 408576698