Title: [227435] trunk/Source/_javascript_Core
Revision
227435
Author
[email protected]
Date
2018-01-23 12:49:26 -0800 (Tue, 23 Jan 2018)

Log Message

Rollout r219636
https://bugs.webkit.org/show_bug.cgi?id=181997
<rdar://problem/35883022>

Unreviewed, as it is a rollout.


* dfg/DFGSpeculativeJIT.cpp:
(JSC::DFG::SpeculativeJIT::emitAllocateRawObject):
* runtime/JSArray.cpp:
(JSC::JSArray::tryCreateUninitializedRestricted):
* runtime/JSArray.h:
(JSC::JSArray::tryCreate):
* runtime/JSObject.cpp:
(JSC::JSObject::ensureLengthSlow):

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (227434 => 227435)


--- trunk/Source/_javascript_Core/ChangeLog	2018-01-23 20:39:34 UTC (rev 227434)
+++ trunk/Source/_javascript_Core/ChangeLog	2018-01-23 20:49:26 UTC (rev 227435)
@@ -1,3 +1,20 @@
+2018-01-23  Robin Morisset  <[email protected]>
+
+        Rollout r219636
+        https://bugs.webkit.org/show_bug.cgi?id=181997
+        <rdar://problem/35883022>
+
+        Unreviewed, as it is a rollout.
+
+        * dfg/DFGSpeculativeJIT.cpp:
+        (JSC::DFG::SpeculativeJIT::emitAllocateRawObject):
+        * runtime/JSArray.cpp:
+        (JSC::JSArray::tryCreateUninitializedRestricted):
+        * runtime/JSArray.h:
+        (JSC::JSArray::tryCreate):
+        * runtime/JSObject.cpp:
+        (JSC::JSObject::ensureLengthSlow):
+
 2018-01-23  Mark Lam  <[email protected]>
 
         Re-arrange TypedArray JSTypes to match the order of the TypedArrayType enum list.

Modified: trunk/Source/_javascript_Core/dfg/DFGSpeculativeJIT.cpp (227434 => 227435)


--- trunk/Source/_javascript_Core/dfg/DFGSpeculativeJIT.cpp	2018-01-23 20:39:34 UTC (rev 227434)
+++ trunk/Source/_javascript_Core/dfg/DFGSpeculativeJIT.cpp	2018-01-23 20:49:26 UTC (rev 227435)
@@ -144,7 +144,7 @@
         slowCases, this, operationNewRawObject, resultGPR, storageGPR,
         structure, vectorLength));
 
-    if (numElements < vectorLength && LIKELY(!hasUndecided(structure->indexingType()))) {
+    if (numElements < vectorLength) {
 #if USE(JSVALUE64)
         if (hasDouble(structure->indexingType()))
             m_jit.move(TrustedImm64(bitwise_cast<int64_t>(PNaN)), scratchGPR);

Modified: trunk/Source/_javascript_Core/runtime/JSArray.cpp (227434 => 227435)


--- trunk/Source/_javascript_Core/runtime/JSArray.cpp	2018-01-23 20:39:34 UTC (rev 227434)
+++ trunk/Source/_javascript_Core/runtime/JSArray.cpp	2018-01-23 20:49:26 UTC (rev 227435)
@@ -91,7 +91,7 @@
         if (hasDouble(indexingType)) {
             for (; i < vectorLength; ++i)
                 butterfly->contiguousDouble().at(std::numeric_limits<uint32_t>::max(), i) = PNaN;
-        } else if (LIKELY(!hasUndecided(indexingType))) {
+        } else {
             for (; i < vectorLength; ++i)
                 butterfly->contiguous().at(std::numeric_limits<uint32_t>::max(), i).clear();
         }

Modified: trunk/Source/_javascript_Core/runtime/JSArray.h (227434 => 227435)


--- trunk/Source/_javascript_Core/runtime/JSArray.h	2018-01-23 20:39:34 UTC (rev 227434)
+++ trunk/Source/_javascript_Core/runtime/JSArray.h	2018-01-23 20:49:26 UTC (rev 227435)
@@ -247,7 +247,7 @@
         butterfly->setPublicLength(initialLength);
         if (hasDouble(indexingType))
             clearArray(butterfly->contiguousDouble().data(), vectorLength);
-        else if (LIKELY(!hasUndecided(indexingType)))
+        else
             clearArray(butterfly->contiguous().data(), vectorLength);
     } else {
         ASSERT(

Modified: trunk/Source/_javascript_Core/runtime/JSObject.cpp (227434 => 227435)


--- trunk/Source/_javascript_Core/runtime/JSObject.cpp	2018-01-23 20:39:34 UTC (rev 227434)
+++ trunk/Source/_javascript_Core/runtime/JSObject.cpp	2018-01-23 20:49:26 UTC (rev 227435)
@@ -3222,7 +3222,7 @@
     if (hasDouble(indexingType())) {
         for (unsigned i = oldVectorLength; i < newVectorLength; ++i)
             butterfly->indexingPayload<double>()[i] = PNaN;
-    } else if (LIKELY(!hasUndecided(indexingType()))) {
+    } else {
         for (unsigned i = oldVectorLength; i < newVectorLength; ++i)
             butterfly->indexingPayload<WriteBarrier<Unknown>>()[i].clear();
     }
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to