Revision: 3784
Author: [email protected]
Date: Wed Feb  3 03:54:57 2010
Log: DebugBreak shouldn't be used when ENABLE_DEBUGGER_SUPPORT isn't defined.
Review URL: http://codereview.chromium.org/568017
http://code.google.com/p/v8/source/detail?r=3784

Modified:
 /branches/bleeding_edge/src/codegen.cc
 /branches/bleeding_edge/src/heap.cc

=======================================
--- /branches/bleeding_edge/src/codegen.cc      Wed Feb  3 00:33:19 2010
+++ /branches/bleeding_edge/src/codegen.cc      Wed Feb  3 03:54:57 2010
@@ -505,11 +505,12 @@
   info()->set_load_stub_cache(value);
 }

-
+#ifdef ENABLE_DEBUGGER_SUPPORT
 void DebuggerStatementStub::Generate(MacroAssembler* masm) {
   Runtime::Function* f = Runtime::FunctionForId(Runtime::kDebugBreak);
   masm->TailCallRuntime(ExternalReference(f), 0, f->result_size);
 }
+#endif


 } }  // namespace v8::internal
=======================================
--- /branches/bleeding_edge/src/heap.cc Wed Feb  3 00:33:19 2010
+++ /branches/bleeding_edge/src/heap.cc Wed Feb  3 03:54:57 2010
@@ -1498,10 +1498,12 @@
 #endif


+#ifdef ENABLE_DEBUGGER_SUPPORT
 void Heap::CreateCEntryDebugBreakStub() {
   DebuggerStatementStub stub;
   set_debugger_statement_code(*stub.GetCode());
 }
+#endif


 void Heap::CreateJSEntryStub() {
@@ -1531,7 +1533,9 @@
   // }
   // To workaround the problem, make separate functions without inlining.
   Heap::CreateCEntryStub();
+#ifdef ENABLE_DEBUGGER_SUPPORT
   Heap::CreateCEntryDebugBreakStub();
+#endif
   Heap::CreateJSEntryStub();
   Heap::CreateJSConstructEntryStub();
 #if V8_TARGET_ARCH_ARM && V8_NATIVE_REGEXP

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

Reply via email to