Title: [120886] trunk/Source/_javascript_Core
Revision
120886
Author
[email protected]
Date
2012-06-20 16:55:18 -0700 (Wed, 20 Jun 2012)

Log Message

Functions on global objects should be specializable
https://bugs.webkit.org/show_bug.cgi?id=89615

Reviewed by Oliver Hunt.
        
I tested to see if this brought back the bug in https://bugs.webkit.org/show_bug.cgi?id=33343,
and it didn't. Bug 33343 was the reason why we disabled global object function specialization
to begin with. So I'm guessing this is safe.

* runtime/JSGlobalObject.cpp:
(JSC::JSGlobalObject::init):

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (120885 => 120886)


--- trunk/Source/_javascript_Core/ChangeLog	2012-06-20 23:49:53 UTC (rev 120885)
+++ trunk/Source/_javascript_Core/ChangeLog	2012-06-20 23:55:18 UTC (rev 120886)
@@ -1,5 +1,19 @@
 2012-06-20  Filip Pizlo  <[email protected]>
 
+        Functions on global objects should be specializable
+        https://bugs.webkit.org/show_bug.cgi?id=89615
+
+        Reviewed by Oliver Hunt.
+        
+        I tested to see if this brought back the bug in https://bugs.webkit.org/show_bug.cgi?id=33343,
+        and it didn't. Bug 33343 was the reason why we disabled global object function specialization
+        to begin with. So I'm guessing this is safe.
+
+        * runtime/JSGlobalObject.cpp:
+        (JSC::JSGlobalObject::init):
+
+2012-06-20  Filip Pizlo  <[email protected]>
+
         build-webkit failure due to illegal 32-bit integer constants in code
         generated by offlineasm
         https://bugs.webkit.org/show_bug.cgi?id=89347

Modified: trunk/Source/_javascript_Core/runtime/JSGlobalObject.cpp (120885 => 120886)


--- trunk/Source/_javascript_Core/runtime/JSGlobalObject.cpp	2012-06-20 23:49:53 UTC (rev 120885)
+++ trunk/Source/_javascript_Core/runtime/JSGlobalObject.cpp	2012-06-20 23:55:18 UTC (rev 120886)
@@ -132,8 +132,6 @@
 {
     ASSERT(JSLock::currentThreadIsHoldingLock());
     
-    structure()->disableSpecificFunctionTracking();
-
     m_globalScopeChain.set(globalData(), this, ScopeChainNode::create(0, this, &globalData(), this, thisValue));
 
     JSGlobalObject::globalExec()->init(0, 0, m_globalScopeChain.get(), CallFrame::noCaller(), 0, 0);
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to