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

Unified Diff: runtime/vm/dart.cc

Issue 2626343002: 1. Added new flag verify_gc_contains and moved all the asserts that do a (Closed)
Patch Set: Created 3 years, 11 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/flag_list.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: runtime/vm/dart.cc
diff --git a/runtime/vm/dart.cc b/runtime/vm/dart.cc
index 42b6cfdb169d605d14c8ad7f84d071603dbe1bdb..57f5cbe759f160cb13e4894006d30ffca83a1c70 100644
--- a/runtime/vm/dart.cc
+++ b/runtime/vm/dart.cc
@@ -139,6 +139,14 @@ char* Dart::InitOnce(const uint8_t* vm_isolate_snapshot,
if (vm_isolate_ != NULL || !Flags::Initialized()) {
return strdup("VM already initialized or flags not initialized.");
}
+#if defined(DEBUG)
+ // Turn on verify_gc_contains if any of the other GC verification flag
+ // is turned on.
+ if (FLAG_verify_before_gc || FLAG_verify_after_gc ||
+ FLAG_verify_on_transition) {
+ FLAG_verify_gc_contains = true;
+ }
+#endif
set_thread_exit_callback(thread_exit);
SetFileCallbacks(file_open, file_read, file_write, file_close);
set_entropy_source_callback(entropy_source);
« no previous file with comments | « no previous file | runtime/vm/flag_list.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698