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

Unified Diff: runtime/vm/compiler.cc

Issue 11014013: Added slow_assert macro and flag for slow development assertions in the VM. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 8 years, 3 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/compiler.cc
diff --git a/runtime/vm/compiler.cc b/runtime/vm/compiler.cc
index 410929ea85931908d34b33bf39d7e0c8b8c88959..26adbdba55d65a75ff1b7932e932fbbdae100ef0 100644
--- a/runtime/vm/compiler.cc
+++ b/runtime/vm/compiler.cc
@@ -5,6 +5,8 @@
#include "vm/compiler.h"
#include "vm/assembler.h"
+
+#include "vm/assert.h"
#include "vm/ast_printer.h"
#include "vm/code_generator.h"
#include "vm/code_patcher.h"
@@ -48,6 +50,7 @@ DEFINE_FLAG(int, deoptimization_counter_threshold, 5,
" certain optimizations");
DEFINE_FLAG(bool, use_inlining, true, "Enable call-site inlining");
DEFINE_FLAG(bool, range_analysis, true, "Enable range analysis");
+DEFINE_FLAG(bool, slow_asserts, false, "Enable slow assertions");
siva 2012/10/01 23:35:40 Ditto comment regarding slow_asserts.
DECLARE_FLAG(bool, print_flow_graph);
@@ -181,8 +184,7 @@ static bool CompileParsedFunctionHelper(const ParsedFunction& parsed_function,
if (FLAG_use_inlining) {
FlowGraphInliner inliner(flow_graph);
inliner.Inline();
- // Verify that the use lists are still valid.
- DEBUG_ASSERT(flow_graph->ValidateUseLists());
+ // Use lists are maintained and validated by the inliner.
}
// Propagate types and eliminate more type tests.

Powered by Google App Engine
This is Rietveld 408576698