Title: [126938] trunk/Source/_javascript_Core
Revision
126938
Author
[email protected]
Date
2012-08-28 16:27:51 -0700 (Tue, 28 Aug 2012)

Log Message

Rolled out r126928, this broke stuff :'-(

* jit/JITPropertyAccess.cpp:
(JSC::JIT::privateCompilePutByIdTransition):
(JSC::JIT::resetPatchPutById):

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (126937 => 126938)


--- trunk/Source/_javascript_Core/ChangeLog	2012-08-28 23:07:34 UTC (rev 126937)
+++ trunk/Source/_javascript_Core/ChangeLog	2012-08-28 23:27:51 UTC (rev 126938)
@@ -1,5 +1,13 @@
 2012-08-28  Gavin Barraclough  <[email protected]>
 
+        Rolled out r126928, this broke stuff :'-(
+
+        * jit/JITPropertyAccess.cpp:
+        (JSC::JIT::privateCompilePutByIdTransition):
+        (JSC::JIT::resetPatchPutById):
+
+2012-08-28  Gavin Barraclough  <[email protected]>
+
         PutById uses DataLabel32, not DataLabelCompact
         https://bugs.webkit.org/show_bug.cgi?id=95245
 

Modified: trunk/Source/_javascript_Core/jit/JITPropertyAccess.cpp (126937 => 126938)


--- trunk/Source/_javascript_Core/jit/JITPropertyAccess.cpp	2012-08-28 23:07:34 UTC (rev 126937)
+++ trunk/Source/_javascript_Core/jit/JITPropertyAccess.cpp	2012-08-28 23:27:51 UTC (rev 126938)
@@ -537,8 +537,6 @@
 
 void JIT::privateCompilePutByIdTransition(StructureStubInfo* stubInfo, Structure* oldStructure, Structure* newStructure, PropertyOffset cachedOffset, StructureChain* chain, ReturnAddressPtr returnAddress, bool direct)
 {
-    peek(regT0, JITSTACKFRAME_ARGS_INDEX);
-
     JumpList failureCases;
     // Check eax is an object of the right Structure.
     failureCases.append(emitJumpIfNotJSCell(regT0));
@@ -1164,7 +1162,7 @@
     else
         repatchBuffer.relink(stubInfo->callReturnLocation, cti_op_put_by_id);
     repatchBuffer.repatch(stubInfo->hotPathBegin.dataLabelPtrAtOffset(stubInfo->patch.baseline.u.put.structureToCompare), reinterpret_cast<void*>(-1));
-    repatchBuffer.repatch(stubInfo->hotPathBegin.dataLabel32AtOffset(stubInfo->patch.baseline.u.put.displacementLabel), 0);
+    repatchBuffer.repatch(stubInfo->hotPathBegin.dataLabelCompactAtOffset(stubInfo->patch.baseline.u.put.displacementLabel), 0);
 }
 
 #endif // USE(JSVALUE64)
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to