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

Unified Diff: runtime/vm/runtime_entry.cc

Issue 2983873002: Exclude StackOverflow stress-testing from PRODUCT mode (Closed)
Patch Set: 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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: runtime/vm/runtime_entry.cc
diff --git a/runtime/vm/runtime_entry.cc b/runtime/vm/runtime_entry.cc
index 322b2bc7f4d4a60531c7080a51f28f5cebb779d5..99d4c2866ef907fbde29e1f4cadf9446910485cb 100644
--- a/runtime/vm/runtime_entry.cc
+++ b/runtime/vm/runtime_entry.cc
@@ -1621,6 +1621,7 @@ DEFINE_RUNTIME_ENTRY(StackOverflow, 0) {
UNREACHABLE();
}
+#if !defined(PRODUCT)
// The following code is used to stress test deoptimization and
// debugger stack tracing.
bool do_deopt = false;
@@ -1678,7 +1679,6 @@ DEFINE_RUNTIME_ENTRY(StackOverflow, 0) {
DeoptimizeFunctionsOnStack();
}
if (do_reload) {
-#ifndef PRODUCT
JSONStream js;
// Maybe adjust the rate of future reloads.
isolate->MaybeIncreaseReloadEveryNStackOverflowChecks();
@@ -1687,9 +1687,7 @@ DEFINE_RUNTIME_ENTRY(StackOverflow, 0) {
if (!success) {
FATAL1("*** Isolate reload failed:\n%s\n", js.ToCString());
}
-#endif
}
-#if !defined(PRODUCT)
if (do_stacktrace) {
String& var_name = String::Handle();
Instance& var_value = Instance::Handle();
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698