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

Unified Diff: runtime/vm/compiler.cc

Issue 10960014: Implement range analysis for smi values. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: cleanup Created 8 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | runtime/vm/flow_graph_optimizer.h » ('j') | runtime/vm/flow_graph_optimizer.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: runtime/vm/compiler.cc
diff --git a/runtime/vm/compiler.cc b/runtime/vm/compiler.cc
index 0095b13eda8e9dcce4c08be3b105496d6c42fb2e..d2135daf7bc414b845c4a179886479cb11eb5c8c 100644
--- a/runtime/vm/compiler.cc
+++ b/runtime/vm/compiler.cc
@@ -205,6 +205,9 @@ static bool CompileParsedFunctionHelper(const ParsedFunction& parsed_function,
if (FLAG_cse) DominatorBasedCSE::Optimize(flow_graph);
if (FLAG_licm) LICM::Optimize(flow_graph);
+ // TODO(vegorov): move optimistic smis unboxing into Sminess propagator.
Florian Schneider 2012/09/21 08:56:44 I don't understand this TODO. Can you make it clea
Vyacheslav Egorov (Google) 2012/09/21 20:08:07 Done.
+ optimizer.InferSmiRanges();
+
// Perform register allocation on the SSA graph.
FlowGraphAllocator allocator(*flow_graph);
allocator.AllocateRegisters();
« no previous file with comments | « no previous file | runtime/vm/flow_graph_optimizer.h » ('j') | runtime/vm/flow_graph_optimizer.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698