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

Unified Diff: runtime/vm/compiler.cc

Issue 9464009: Add enough compiler support to compile empty functions on x64. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Include file inadvertently left out. Created 8 years, 10 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: runtime/vm/compiler.cc
diff --git a/runtime/vm/compiler.cc b/runtime/vm/compiler.cc
index 48c61022e008503fc3f5acddf15e398e02cb5ea2..96e665f4aa7ecff610f380f64d6fdcca8dfa61a6 100644
--- a/runtime/vm/compiler.cc
+++ b/runtime/vm/compiler.cc
@@ -14,6 +14,7 @@
#include "vm/exceptions.h"
#include "vm/flags.h"
#include "vm/flow_graph_builder.h"
+#include "vm/flow_graph_compiler.h"
#include "vm/longjump.h"
#include "vm/object.h"
#include "vm/object_store.h"
@@ -129,6 +130,16 @@ static RawError* CompileFunctionHelper(const Function& function,
if (setjmp(*bailout_jump.Set()) == 0) {
FlowGraphBuilder graph_builder(parsed_function);
graph_builder.BuildGraph();
+
+ // Try to compile on x64 (only for now).
+#ifdef TARGET_ARCH_X64
+ Assembler assembler;
+ FlowGraphCompiler graph_compiler(&assembler,
+ parsed_function,
+ graph_builder.blocks());
+ graph_compiler.CompileGraph();
srdjan 2012/02/24 23:13:02 It would be better to implement "dummy" FlowGraphC
Kevin Millikin (Google) 2012/02/27 08:22:07 Agreed. Done.
+#endif
+
} else {
// We bailed out.
Error& bailout_error = Error::Handle(

Powered by Google App Engine
This is Rietveld 408576698