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

Unified Diff: runtime/vm/compiler.cc

Issue 10831261: Build and use stack maps in the SSA compiler. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 8 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
« no previous file with comments | « no previous file | runtime/vm/flow_graph_allocator.h » ('j') | runtime/vm/flow_graph_allocator.h » ('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 e4c0c6b8df240c354e9c574df11d2ca90209d6f4..2fc5d7f07ba2543365130922941988619c3e4fc8 100644
--- a/runtime/vm/compiler.cc
+++ b/runtime/vm/compiler.cc
@@ -216,6 +216,9 @@ static bool CompileParsedFunctionHelper(
const Function& function = parsed_function.function();
const Code& code = Code::Handle(Code::FinalizeCode(function, &assembler));
code.set_is_optimized(optimized);
+ if (optimized && use_ssa) {
+ code.set_spill_slot_count(graph_compiler.StackSize());
+ }
graph_compiler.FinalizePcDescriptors(code);
graph_compiler.FinalizeDeoptInfo(code);
graph_compiler.FinalizeStackmaps(code);
@@ -312,6 +315,7 @@ static RawError* CompileFunctionHelper(const Function& function,
start + instructions.size(),
code.comments());
OS::Print("}\n");
+
OS::Print("Pointer offsets for function: {\n");
// Pointer offsets are stored in descending order.
for (intptr_t i = code.pointer_offsets_length() - 1; i >= 0; i--) {
@@ -322,12 +326,13 @@ static RawError* CompileFunctionHelper(const Function& function,
code.GetPointerOffsetAt(i), addr, obj.ToCString());
}
OS::Print("}\n");
+
OS::Print("PC Descriptors for function '%s' {\n", function_fullname);
- OS::Print("(pc\t\tkind\t\tid\ttok-ix\ttry/deopt-ix)\n");
+ PcDescriptors::PrintHeaderString();
const PcDescriptors& descriptors =
PcDescriptors::Handle(code.pc_descriptors());
- OS::Print("%s\n", descriptors.ToCString());
- OS::Print("}\n");
+ OS::Print("%s\n}\n", descriptors.ToCString());
+
const Array& deopt_info_array = Array::Handle(code.deopt_info_array());
if (deopt_info_array.Length() > 0) {
OS::Print("DeoptInfo: {\n");
@@ -337,6 +342,7 @@ static RawError* CompileFunctionHelper(const Function& function,
}
OS::Print("}\n");
}
+
const Array& object_table = Array::Handle(code.object_table());
if (object_table.Length() > 0) {
OS::Print("Object Table: {\n");
@@ -346,6 +352,18 @@ static RawError* CompileFunctionHelper(const Function& function,
}
OS::Print("}\n");
}
+
+ OS::Print("Stackmaps for function '%s' {\n", function_fullname);
+ if (code.stackmaps() != Array::null()) {
+ const Array& stackmap_table = Array::Handle(code.stackmaps());
+ Stackmap& map = Stackmap::Handle();
+ for (intptr_t i = 0; i < stackmap_table.Length(); ++i) {
+ map ^= stackmap_table.At(i);
+ OS::Print("%s\n", map.ToCString());
+ }
+ }
+ OS::Print("}\n");
+
OS::Print("Variable Descriptors for function '%s' {\n",
function_fullname);
const LocalVarDescriptors& var_descriptors =
« no previous file with comments | « no previous file | runtime/vm/flow_graph_allocator.h » ('j') | runtime/vm/flow_graph_allocator.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698