Changeset 34374 in webkit for trunk/JavaScriptCore

Timestamp:

Jun 5, 2008, 3:30:06 AM (17 years ago)

Author:

[email protected]

Message:

2008-06-05 Maciej Stachowiak <​[email protected]>

Reviewed by Alexey and Oliver.

• remove profiler fetch hack I measure an 0.5% progression from this, others show a wash. It seems not needed any more.

• VM/Machine.cpp: (KJS::Machine::privateExecute):

Location:

trunk/JavaScriptCore

Files:

• ChangeLog (modified) (1 diff)
• VM/Machine.cpp (modified) (1 diff)

trunk/JavaScriptCore/ChangeLog

@@ +1 @@
+2008-06-05  Maciej Stachowiak  <[email protected]>
+
+        Reviewed by Alexey and Oliver.
+        
+        - remove profiler fetch hack
+        I measure an 0.5% progression from this, others show a wash. It seems not needed any more.
+
+        * VM/Machine.cpp:
+        (KJS::Machine::privateExecute):
+
 2008-06-05  Cameron Zwarich  <[email protected]>
 

trunk/JavaScriptCore/VM/Machine.cpp

@@ -891 +891 @@
     Profiler** enabledProfilerReference = Profiler::enabledProfilerReference();
 
-#if HAVE(COMPUTED_GOTO)
-    // Yet another hack around GCC's various foibles, in this case fetching the
-    // profiler reference results in a regression.  Removing this indirection
-    // results in a 0.8% regression.
-    goto *(&&profilerFetchHack);
-    profilerFetchHack:
-#endif
-
     registerFile->setSafeForReentry(false);
 #define VM_CHECK_EXCEPTION() \