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

Unified Diff: runtime/vm/flow_graph_compiler_arm64.cc

Issue 335443002: Setup R10/EDX to be valid Oops before stub calls (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 6 years, 6 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/flow_graph_compiler_arm64.cc
diff --git a/runtime/vm/flow_graph_compiler_arm64.cc b/runtime/vm/flow_graph_compiler_arm64.cc
index 66a29de93087e3b63b3ebc13b34bfd50bf88e378..1530610342b4847e6fb70b6a134d39ac9c19aee5 100644
--- a/runtime/vm/flow_graph_compiler_arm64.cc
+++ b/runtime/vm/flow_graph_compiler_arm64.cc
@@ -1215,6 +1215,7 @@ void FlowGraphCompiler::EmitInstanceCall(ExternalLabel* target_label,
intptr_t token_pos,
LocationSummary* locs) {
ASSERT(Array::Handle(ic_data.arguments_descriptor()).Length() > 0);
+ __ LoadImmediate(R4, 0, kNoPP);
__ LoadObject(R5, ic_data, PP);
GenerateDartCall(deopt_id,
token_pos,
@@ -1222,6 +1223,9 @@ void FlowGraphCompiler::EmitInstanceCall(ExternalLabel* target_label,
PcDescriptors::kIcCall,
locs);
__ Drop(argument_count);
+#if defined(DEBUG)
+ __ LoadImmediate(R4, 1, kNoPP); // Garbage that looks like an object.
+#endif
}
@@ -1301,12 +1305,16 @@ void FlowGraphCompiler::EmitUnoptimizedStaticCall(
UNIMPLEMENTED();
}
ExternalLabel target_label(label_address);
+ __ LoadImmediate(R4, 0, kNoPP);
__ LoadObject(R5, ic_data, PP);
GenerateDartCall(deopt_id,
token_pos,
&target_label,
PcDescriptors::kUnoptStaticCall,
locs);
+#if defined(DEBUG)
+ __ LoadImmediate(R5, 1, kNoPP); // Garbage that looks like an object.
+#endif
__ Drop(argument_count);
}
@@ -1371,6 +1379,8 @@ void FlowGraphCompiler::EmitEqualityRegRegCompare(Register left,
__ BranchLinkPatchable(
&StubCode::OptimizedIdenticalWithNumberCheckLabel());
} else {
+ __ LoadImmediate(R4, 0, kNoPP);
+ __ LoadImmediate(R5, 0, kNoPP);
__ BranchLinkPatchable(
&StubCode::UnoptimizedIdenticalWithNumberCheckLabel());
}
@@ -1379,6 +1389,13 @@ void FlowGraphCompiler::EmitEqualityRegRegCompare(Register left,
Isolate::kNoDeoptId,
token_pos);
}
+#if defined(DEBUG)
+ if (!is_optimizing()) {
+ // Do this *after* adding the pc descriptor!
+ __ LoadImmediate(R4, 1, kNoPP); // Garbage that looks like an object.
+ __ LoadImmediate(R5, 1, kNoPP);
+ }
+#endif
// Stub returns result in flags (result of a cmpl, we need ZF computed).
__ Pop(right);
__ Pop(left);

Powered by Google App Engine
This is Rietveld 408576698