Title: [188086] trunk/Source/_javascript_Core
Revision
188086
Author
commit-qu...@webkit.org
Date
2015-08-06 16:25:46 -0700 (Thu, 06 Aug 2015)

Log Message

The typedArrayLength function in FTLLowerDFGToLLVM is dead code.
https://bugs.webkit.org/show_bug.cgi?id=147749

Patch by Keith Miller <keith_mil...@apple.com> on 2015-08-06
Reviewed by Filip Pizlo.

Removed dead code elimination. the TypedArray length is compiled in compileGetArrayLength()
thus no one calls this code.

* ftl/FTLLowerDFGToLLVM.cpp:
(JSC::FTL::DFG::LowerDFGToLLVM::typedArrayLength): Deleted.

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (188085 => 188086)


--- trunk/Source/_javascript_Core/ChangeLog	2015-08-06 23:23:56 UTC (rev 188085)
+++ trunk/Source/_javascript_Core/ChangeLog	2015-08-06 23:25:46 UTC (rev 188086)
@@ -1,5 +1,18 @@
 2015-08-06  Keith Miller  <keith_mil...@apple.com>
 
+        The typedArrayLength function in FTLLowerDFGToLLVM is dead code.
+        https://bugs.webkit.org/show_bug.cgi?id=147749
+
+        Reviewed by Filip Pizlo.
+
+        Removed dead code elimination. the TypedArray length is compiled in compileGetArrayLength()
+        thus no one calls this code.
+
+        * ftl/FTLLowerDFGToLLVM.cpp:
+        (JSC::FTL::DFG::LowerDFGToLLVM::typedArrayLength): Deleted.
+
+2015-08-06  Keith Miller  <keith_mil...@apple.com>
+
         The JSONP parser incorrectly parsers -0 as +0.
         https://bugs.webkit.org/show_bug.cgi?id=147590
 

Modified: trunk/Source/_javascript_Core/ftl/FTLLowerDFGToLLVM.cpp (188085 => 188086)


--- trunk/Source/_javascript_Core/ftl/FTLLowerDFGToLLVM.cpp	2015-08-06 23:23:56 UTC (rev 188085)
+++ trunk/Source/_javascript_Core/ftl/FTLLowerDFGToLLVM.cpp	2015-08-06 23:25:46 UTC (rev 188086)
@@ -6047,19 +6047,6 @@
             m_out.phi(m_out.intPtr, fastButterfly, slowButterfly));
     }
     
-    LValue typedArrayLength(Edge baseEdge, ArrayMode arrayMode, LValue base)
-    {
-        JSArrayBufferView* view = m_graph.tryGetFoldableView(provenValue(baseEdge), arrayMode);
-        if (view)
-            return m_out.constInt32(view->length());
-        return m_out.load32NonNegative(base, m_heaps.JSArrayBufferView_length);
-    }
-    
-    LValue typedArrayLength(Edge baseEdge, ArrayMode arrayMode)
-    {
-        return typedArrayLength(baseEdge, arrayMode, lowCell(baseEdge));
-    }
-    
     LValue boolify(Edge edge)
     {
         switch (edge.useKind()) {
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to