Title: [179241] trunk/Source/_javascript_Core
Revision
179241
Author
[email protected]
Date
2015-01-27 21:48:59 -0800 (Tue, 27 Jan 2015)

Log Message

ExitSiteData saying m_takesSlowPath shouldn't mean early returning takesSlowPath() since for the non-LLInt case we later set m_couldTakeSlowPath, which is more precise
https://bugs.webkit.org/show_bug.cgi?id=140980

Reviewed by Oliver Hunt.

* bytecode/CallLinkStatus.cpp:
(JSC::CallLinkStatus::computeFor):

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (179240 => 179241)


--- trunk/Source/_javascript_Core/ChangeLog	2015-01-28 05:29:09 UTC (rev 179240)
+++ trunk/Source/_javascript_Core/ChangeLog	2015-01-28 05:48:59 UTC (rev 179241)
@@ -1,5 +1,15 @@
 2015-01-27  Filip Pizlo  <[email protected]>
 
+        ExitSiteData saying m_takesSlowPath shouldn't mean early returning takesSlowPath() since for the non-LLInt case we later set m_couldTakeSlowPath, which is more precise
+        https://bugs.webkit.org/show_bug.cgi?id=140980
+
+        Reviewed by Oliver Hunt.
+
+        * bytecode/CallLinkStatus.cpp:
+        (JSC::CallLinkStatus::computeFor):
+
+2015-01-27  Filip Pizlo  <[email protected]>
+
         Move DFGBinarySwitch out of the DFG so that all of the JITs can use it
         https://bugs.webkit.org/show_bug.cgi?id=140959
 

Modified: trunk/Source/_javascript_Core/bytecode/CallLinkStatus.cpp (179240 => 179241)


--- trunk/Source/_javascript_Core/bytecode/CallLinkStatus.cpp	2015-01-28 05:29:09 UTC (rev 179240)
+++ trunk/Source/_javascript_Core/bytecode/CallLinkStatus.cpp	2015-01-28 05:48:59 UTC (rev 179241)
@@ -86,12 +86,13 @@
     UNUSED_PARAM(map);
 #if ENABLE(DFG_JIT)
     ExitSiteData exitSiteData = computeExitSiteData(locker, profiledBlock, bytecodeIndex);
-    if (exitSiteData.m_takesSlowPath)
-        return takesSlowPath();
     
     CallLinkInfo* callLinkInfo = map.get(CodeOrigin(bytecodeIndex));
-    if (!callLinkInfo)
+    if (!callLinkInfo) {
+        if (exitSiteData.m_takesSlowPath)
+            return takesSlowPath();
         return computeFromLLInt(locker, profiledBlock, bytecodeIndex);
+    }
     
     return computeFor(locker, profiledBlock, *callLinkInfo, exitSiteData);
 #else
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to