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

Unified Diff: runtime/vm/kernel_to_il.cc

Issue 2981173002: Remove the debugger_ field from Isolate in a PRODUCT build (Closed)
Patch Set: Remove flag support_debugger 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/isolate.cc ('k') | runtime/vm/object.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: runtime/vm/kernel_to_il.cc
diff --git a/runtime/vm/kernel_to_il.cc b/runtime/vm/kernel_to_il.cc
index 4ada7539e9aa809197f4222a06d763f7ebde1227..a584c0f4022cb06a012a68ea8ef480a8909fae1e 100644
--- a/runtime/vm/kernel_to_il.cc
+++ b/runtime/vm/kernel_to_il.cc
@@ -1850,15 +1850,19 @@ Fragment FlowGraphBuilder::CheckVariableTypeInCheckedMode(
bool FlowGraphBuilder::NeedsDebugStepCheck(const Function& function,
TokenPosition position) {
- return FLAG_support_debugger && position.IsDebugPause() &&
- !function.is_native() && function.is_debuggable();
+ return position.IsDebugPause() && !function.is_native() &&
+ function.is_debuggable();
}
bool FlowGraphBuilder::NeedsDebugStepCheck(Value* value,
TokenPosition position) {
- if (!FLAG_support_debugger || !position.IsDebugPause()) return false;
+ if (!position.IsDebugPause()) {
+ return false;
+ }
Definition* definition = value->definition();
- if (definition->IsConstant() || definition->IsLoadStaticField()) return true;
+ if (definition->IsConstant() || definition->IsLoadStaticField()) {
+ return true;
+ }
if (definition->IsAllocateObject()) {
return !definition->AsAllocateObject()->closure_function().IsNull();
}
« no previous file with comments | « runtime/vm/isolate.cc ('k') | runtime/vm/object.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698