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

Unified Diff: runtime/vm/debugger.cc

Issue 2976723003: Eliminate dependencies on assemblers and code stubs in precompiled runtime. (Closed)
Patch Set: Eliminate precompiled runtime flag Created 3 years, 5 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 | « runtime/vm/debugger.h ('k') | runtime/vm/deferred_objects.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: runtime/vm/debugger.cc
diff --git a/runtime/vm/debugger.cc b/runtime/vm/debugger.cc
index 6bfbede60abb9d9e5554b8acfd88fed330125959..39ba697eb85b2b3a34d824adfd17943785e36d93 100644
--- a/runtime/vm/debugger.cc
+++ b/runtime/vm/debugger.cc
@@ -720,8 +720,10 @@ void ActivationFrame::PrintDescriptorsError(const char* message) {
OS::PrintErr("pc_ %" Px "\n", pc_);
OS::PrintErr("deopt_id_ %" Px "\n", deopt_id_);
OS::PrintErr("context_level_ %" Px "\n", context_level_);
+#if !defined(DART_PRECOMPILED_RUNTIME)
DisassembleToStdout formatter;
code().Disassemble(&formatter);
+#endif // !defined(DART_PRECOMPILED_RUNTIME)
PcDescriptors::Handle(code().pc_descriptors()).Print();
StackFrameIterator frames(StackFrameIterator::kDontValidateFrames,
Thread::Current(),
@@ -1890,6 +1892,7 @@ ActivationFrame* Debugger::CollectDartFrame(Isolate* isolate,
}
+#if !defined(DART_PRECOMPILED_RUNTIME)
RawArray* Debugger::DeoptimizeToArray(Thread* thread,
StackFrame* frame,
const Code& code) {
@@ -1911,6 +1914,7 @@ RawArray* Debugger::DeoptimizeToArray(Thread* thread,
return dest_frame.raw();
}
+#endif // !defined(DART_PRECOMPILED_RUNTIME)
DebuggerStackTrace* Debugger::CollectStackTrace() {
@@ -1949,7 +1953,8 @@ void Debugger::AppendCodeFrames(Thread* thread,
Code* code,
Code* inlined_code,
Array* deopt_frame) {
- if (code->is_optimized() && !FLAG_precompiled_runtime) {
+#if !defined(DART_PRECOMPILED_RUNTIME)
+ if (code->is_optimized()) {
// TODO(rmacnak): Use CodeSourceMap
*deopt_frame = DeoptimizeToArray(thread, frame, *code);
for (InlinedFunctionsIterator it(*code, frame->pc()); !it.Done();
@@ -1966,10 +1971,11 @@ void Debugger::AppendCodeFrames(Thread* thread,
*inlined_code, *deopt_frame,
deopt_frame_offset));
}
- } else {
- stack_trace->AddActivation(CollectDartFrame(
- isolate, frame->pc(), frame, *code, Object::null_array(), 0));
+ return;
}
+#endif // !defined(DART_PRECOMPILED_RUNTIME)
+ stack_trace->AddActivation(CollectDartFrame(isolate, frame->pc(), frame,
+ *code, Object::null_array(), 0));
}
@@ -2060,11 +2066,14 @@ DebuggerStackTrace* Debugger::CollectAsyncCausalStackTrace() {
DebuggerStackTrace* Debugger::CollectAwaiterReturnStackTrace() {
+#if defined(DART_PRECOMPILED_RUNTIME)
+ // Causal async stacks are not supported in the AOT runtime.
+ ASSERT(!FLAG_async_debugger);
+ return NULL;
+#else
if (!FLAG_async_debugger) {
return NULL;
}
- // Causal async stacks are not supported in the AOT runtime.
- ASSERT(!FLAG_precompiled_runtime);
Thread* thread = Thread::Current();
Zone* zone = thread->zone();
@@ -2203,6 +2212,7 @@ DebuggerStackTrace* Debugger::CollectAwaiterReturnStackTrace() {
}
return stack_trace;
+#endif // defined(DART_PRECOMPILED_RUNTIME)
}
« no previous file with comments | « runtime/vm/debugger.h ('k') | runtime/vm/deferred_objects.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698