Changeset 194383 in webkit

Timestamp:

Dec 22, 2015, 6:33:48 PM (10 years ago)

Author:

fpizlo@apple.com

Message:

FTL B3 should use the right type for comparison slow paths
​https://bugs.webkit.org/show_bug.cgi?id=152521

Reviewed by Saam Barati.

Fixes a small goof that was leading to B3 validation failures.

• ftl/FTLLowerDFGToLLVM.cpp:

(JSC::FTL::DFG::LowerDFGToLLVM::nonSpeculativeCompare):

Location:

trunk/Source/JavaScriptCore

Files:

• ChangeLog (modified) (1 diff)
• ftl/FTLLowerDFGToLLVM.cpp (modified) (1 diff)

trunk/Source/JavaScriptCore/ChangeLog

@@ +1 @@
+2015-12-22  Filip Pizlo  <fpizlo@apple.com>
+
+        FTL B3 should use the right type for comparison slow paths
+        https://bugs.webkit.org/show_bug.cgi?id=152521
+
+        Reviewed by Saam Barati.
+
+        Fixes a small goof that was leading to B3 validation failures.
+
+        * ftl/FTLLowerDFGToLLVM.cpp:
+        (JSC::FTL::DFG::LowerDFGToLLVM::nonSpeculativeCompare):
+
 2015-12-22  Filip Pizlo  <fpizlo@apple.com>
 

trunk/Source/JavaScriptCore/ftl/FTLLowerDFGToLLVM.cpp

@@ -7368 +7368 @@
         m_out.appendTo(slowPath, continuation);
         ValueFromBlock slowResult = m_out.anchor(m_out.notNull(vmCall(
-            m_out.boolean, m_out.operation(helperFunction), m_callFrame, left, right)));
+            m_out.intPtr, m_out.operation(helperFunction), m_callFrame, left, right)));
         m_out.jump(continuation);