Revision: 10480
Author:   [email protected]
Date:     Mon Jan 23 17:46:26 2012
Log:      Put a slow assert under the enable_slow_asserts flag.

Otherwise really large functions take too long to compile
in debug mode.
Review URL: http://codereview.chromium.org/9212040
http://code.google.com/p/v8/source/detail?r=10480

Modified:
 /branches/bleeding_edge/src/full-codegen.cc

=======================================
--- /branches/bleeding_edge/src/full-codegen.cc Fri Jan 13 05:09:52 2012
+++ /branches/bleeding_edge/src/full-codegen.cc Mon Jan 23 17:46:26 2012
@@ -370,12 +370,14 @@
       StateField::encode(state) | PcField::encode(masm_->pc_offset());
   BailoutEntry entry = { id, pc_and_state };
 #ifdef DEBUG
-  // Assert that we don't have multiple bailout entries for the same node.
-  for (int i = 0; i < bailout_entries_.length(); i++) {
-    if (bailout_entries_.at(i).id == entry.id) {
-      AstPrinter printer;
-      PrintF("%s", printer.PrintProgram(info_->function()));
-      UNREACHABLE();
+  if (FLAG_enable_slow_asserts) {
+ // Assert that we don't have multiple bailout entries for the same node.
+    for (int i = 0; i < bailout_entries_.length(); i++) {
+      if (bailout_entries_.at(i).id == entry.id) {
+        AstPrinter printer;
+        PrintF("%s", printer.PrintProgram(info_->function()));
+        UNREACHABLE();
+      }
     }
   }
 #endif  // DEBUG

--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to