Revision: 11398
Author:   [email protected]
Date:     Fri Apr 20 03:43:02 2012
Log:      Relax test-debug/ThreadedDebugging to stop flakiness.

Actual bug tracked as issue 2047.

[email protected]

Review URL: https://chromiumcodereview.appspot.com/10167002
http://code.google.com/p/v8/source/detail?r=11398

Modified:
 /branches/bleeding_edge/test/cctest/test-debug.cc

=======================================
--- /branches/bleeding_edge/test/cctest/test-debug.cc Thu Mar 8 02:21:43 2012 +++ /branches/bleeding_edge/test/cctest/test-debug.cc Fri Apr 20 03:43:02 2012
@@ -5013,7 +5013,10 @@
   if (IsBreakEventMessage(print_buffer)) {
     // Check that we are inside the while loop.
     int source_line = GetSourceLineFromBreakEventMessage(print_buffer);
-    CHECK(8 <= source_line && source_line <= 13);
+    // TODO(2047): This should really be 8 <= source_line <= 13; but we
+    // currently have an off-by-one error when calculating the source
+    // position corresponding to the program counter at the debug break.
+    CHECK(7 <= source_line && source_line <= 13);
     threaded_debugging_barriers.barrier_2.Wait();
   }
 }

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

Reply via email to