Changeset 34842 in webkit for trunk/JavaScriptCore/VM/Machine.h

Timestamp:

Jun 27, 2008, 9:02:03 PM (17 years ago)

Author:

[email protected]

Message:

Bug 18626: SQUIRRELFISH: support the "slow script" dialog <​https://bugs.webkit.org/show_bug.cgi?id=18626>
<rdar://problem/5973931> Slow script dialog needs to be reimplemented for squirrelfish

Reviewed by Sam

Adds support for the slow script dialog in squirrelfish. This requires the addition
of three new op codes, op_loop, op_loop_if_true, and op_loop_if_less which have the
same behaviour as their simple jump equivalents but have an additional time out check.

Additional assertions were added to other jump instructions to prevent accidentally
creating loops with jump types that do not support time out checks.

Sunspider does not report a regression, however this appears very sensitive to code
layout and hardware, so i would expect up to a 1% regression on other systems.

Part of this required moving the old timeout logic from JSGlobalObject and into Machine
which is the cause of a number of the larger diff blocks.

File:

• trunk/JavaScriptCore/VM/Machine.h (modified) (3 diffs)

trunk/JavaScriptCore/VM/Machine.h

@@ -43 +43 @@
     class Instruction;
     class JSFunction;
+    class JSGlobalObject;
     class ProgramNode;
     class Register;
@@ -97 +98 @@
 
         void getFunctionAndArguments(Register** registerBase, Register* callFrame, JSFunction*&, Register*& argv, int& argc);
+        void setTimeoutTime(unsigned timeoutTime) { m_timeoutTime = timeoutTime; }
+        
+        void startTimeoutCheck()
+        {
+            if (!m_timeoutCheckCount)
+                resetTimeoutCheck();
+            
+            ++m_timeoutCheckCount;
+        }
+        
+        void stopTimeoutCheck()
+        {
+            --m_timeoutCheckCount;
+        }
 
+        inline void initTimeout()
+        {
+            resetTimeoutCheck();
+            m_timeoutTime = 0;
+            m_timeoutCheckCount = 0;
+        }
         void mark(Heap* heap) { m_registerFile.mark(heap); }
 
@@ -119 +140 @@
         void dumpRegisters(const CodeBlock*, RegisterFile*, const Register*);
 
+        JSValue* checkTimeout(JSGlobalObject*);
+        void resetTimeoutCheck();
+
         int m_reentryDepth;
+        unsigned m_timeoutTime;
+        unsigned m_timeAtLastCheckTimeout;
+        unsigned m_timeExecuting;
+        unsigned m_timeoutCheckCount;
+        unsigned m_ticksUntilNextTimeoutCheck;
+
         RegisterFile m_registerFile;