Title: [223729] trunk/Source/_javascript_Core
Revision
223729
Author
sbar...@apple.com
Date
2017-10-19 16:49:23 -0700 (Thu, 19 Oct 2017)

Log Message

REGRESSION(r223691): DFGByteCodeParser.cpp:1483:83: warning: comparison is always false due to limited range of data type [-Wtype-limits]
https://bugs.webkit.org/show_bug.cgi?id=178543

Reviewed by Filip Pizlo.

* dfg/DFGByteCodeParser.cpp:
(JSC::DFG::ByteCodeParser::handleRecursiveTailCall):

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (223728 => 223729)


--- trunk/Source/_javascript_Core/ChangeLog	2017-10-19 23:48:45 UTC (rev 223728)
+++ trunk/Source/_javascript_Core/ChangeLog	2017-10-19 23:49:23 UTC (rev 223729)
@@ -1,5 +1,15 @@
 2017-10-19  Saam Barati  <sbar...@apple.com>
 
+        REGRESSION(r223691): DFGByteCodeParser.cpp:1483:83: warning: comparison is always false due to limited range of data type [-Wtype-limits]
+        https://bugs.webkit.org/show_bug.cgi?id=178543
+
+        Reviewed by Filip Pizlo.
+
+        * dfg/DFGByteCodeParser.cpp:
+        (JSC::DFG::ByteCodeParser::handleRecursiveTailCall):
+
+2017-10-19  Saam Barati  <sbar...@apple.com>
+
         re-inline ObjectAllocationProfile::initializeProfile
         https://bugs.webkit.org/show_bug.cgi?id=178532
 

Modified: trunk/Source/_javascript_Core/dfg/DFGByteCodeParser.cpp (223728 => 223729)


--- trunk/Source/_javascript_Core/dfg/DFGByteCodeParser.cpp	2017-10-19 23:48:45 UTC (rev 223728)
+++ trunk/Source/_javascript_Core/dfg/DFGByteCodeParser.cpp	2017-10-19 23:49:23 UTC (rev 223729)
@@ -1480,7 +1480,7 @@
         addJumpTo(*entryBlockPtr);
         return true;
         // It would be unsound to jump over a non-tail call: the "tail" call is not really a tail call in that case.
-    } while (stackEntry->m_inlineCallFrame && stackEntry->m_inlineCallFrame->kind == TailCall && (stackEntry = stackEntry->m_caller));
+    } while (stackEntry->m_inlineCallFrame && stackEntry->m_inlineCallFrame->kind == InlineCallFrame::TailCall && (stackEntry = stackEntry->m_caller));
 
     // The tail call was not recursive
     return false;
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to