Title: [161353] trunk/Source/_javascript_Core
Revision
161353
Author
[email protected]
Date
2014-01-06 10:56:26 -0800 (Mon, 06 Jan 2014)

Log Message

LLInt shouldnt check for ENABLE(_javascript__DEBUGGER).

Rubber stamped by Mark Hahnenberg.

* debugger/Debugger.h:
(JSC::Debugger::Debugger):
* llint/LLIntOfflineAsmConfig.h:
* llint/LowLevelInterpreter.asm:

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (161352 => 161353)


--- trunk/Source/_javascript_Core/ChangeLog	2014-01-06 18:33:34 UTC (rev 161352)
+++ trunk/Source/_javascript_Core/ChangeLog	2014-01-06 18:56:26 UTC (rev 161353)
@@ -1,3 +1,14 @@
+2014-01-06  Filip Pizlo  <[email protected]>
+
+        LLInt shouldnt check for ENABLE(_javascript__DEBUGGER).
+
+        Rubber stamped by Mark Hahnenberg.
+
+        * debugger/Debugger.h:
+        (JSC::Debugger::Debugger):
+        * llint/LLIntOfflineAsmConfig.h:
+        * llint/LowLevelInterpreter.asm:
+
 2014-01-05  Sam Weinig  <[email protected]>
 
         [JS] Implement Promise.race()

Modified: trunk/Source/_javascript_Core/debugger/Debugger.h (161352 => 161353)


--- trunk/Source/_javascript_Core/debugger/Debugger.h	2014-01-06 18:33:34 UTC (rev 161352)
+++ trunk/Source/_javascript_Core/debugger/Debugger.h	2014-01-06 18:56:26 UTC (rev 161353)
@@ -195,7 +195,7 @@
 
 class Debugger {
 public:
-    Debugger(bool = false) { }
+    Debugger(bool = false) : m_needsOpDebugCallbacks(false) { }
     bool needsOpDebugCallbacks() const { return false; }
     bool needsExceptionCallbacks() const { return false; }
     void detach(JSGlobalObject*) { }
@@ -207,6 +207,8 @@
     void willExecuteProgram(CallFrame*) { }
     void didExecuteProgram(CallFrame*) { }
     void didReachBreakpoint(CallFrame*) { }
+
+    bool m_needsOpDebugCallbacks;
 };
 
 #endif // ENABLE(_javascript__DEBUGGER)

Modified: trunk/Source/_javascript_Core/llint/LLIntOfflineAsmConfig.h (161352 => 161353)


--- trunk/Source/_javascript_Core/llint/LLIntOfflineAsmConfig.h	2014-01-06 18:33:34 UTC (rev 161352)
+++ trunk/Source/_javascript_Core/llint/LLIntOfflineAsmConfig.h	2014-01-06 18:56:26 UTC (rev 161353)
@@ -154,12 +154,6 @@
 #define OFFLINE_ASM_ALWAYS_ALLOCATE_SLOW 0
 #endif
 
-#if ENABLE(_javascript__DEBUGGER)
-#define OFFLINE_ASM_JAVASCRIPT_DEBUGGER 1
-#else
-#define OFFLINE_ASM_JAVASCRIPT_DEBUGGER 0
-#endif
-
 #if ENABLE(VALUE_PROFILER)
 #define OFFLINE_ASM_VALUE_PROFILER 1
 #else

Modified: trunk/Source/_javascript_Core/llint/LowLevelInterpreter.asm (161352 => 161353)


--- trunk/Source/_javascript_Core/llint/LowLevelInterpreter.asm	2014-01-06 18:33:34 UTC (rev 161352)
+++ trunk/Source/_javascript_Core/llint/LowLevelInterpreter.asm	2014-01-06 18:56:26 UTC (rev 161353)
@@ -826,7 +826,6 @@
 
 
 _llint_op_debug:
-if _javascript__DEBUGGER
     traceExecution()
     loadp CodeBlock[cfr], t0
     loadp CodeBlock::m_globalObject[t0], t0
@@ -837,7 +836,6 @@
 
     callSlowPath(_llint_slow_path_debug)
 .opDebugDone:                    
-end
     dispatch(2)
 
 
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to