Title: [127625] trunk/Source/_javascript_Core
Revision
127625
Author
[email protected]
Date
2012-09-05 12:07:35 -0700 (Wed, 05 Sep 2012)

Log Message

Remove use of JSCell::classInfoOffset() from virtualForThunkGenerator
https://bugs.webkit.org/show_bug.cgi?id=95821

Reviewed by Oliver Hunt.

We can replace the load of the ClassInfo from the object with a load from the Structure.

* dfg/DFGThunks.cpp:
(JSC::DFG::virtualForThunkGenerator):

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (127624 => 127625)


--- trunk/Source/_javascript_Core/ChangeLog	2012-09-05 19:06:49 UTC (rev 127624)
+++ trunk/Source/_javascript_Core/ChangeLog	2012-09-05 19:07:35 UTC (rev 127625)
@@ -1,3 +1,15 @@
+2012-09-05  Mark Hahnenberg  <[email protected]>
+
+        Remove use of JSCell::classInfoOffset() from virtualForThunkGenerator
+        https://bugs.webkit.org/show_bug.cgi?id=95821
+
+        Reviewed by Oliver Hunt.
+
+        We can replace the load of the ClassInfo from the object with a load from the Structure.
+
+        * dfg/DFGThunks.cpp:
+        (JSC::DFG::virtualForThunkGenerator):
+
 2012-09-05  Benjamin Poulain  <[email protected]>
 
         Fix the uses of String::operator+=() for Mac

Modified: trunk/Source/_javascript_Core/dfg/DFGThunks.cpp (127624 => 127625)


--- trunk/Source/_javascript_Core/dfg/DFGThunks.cpp	2012-09-05 19:06:49 UTC (rev 127624)
+++ trunk/Source/_javascript_Core/dfg/DFGThunks.cpp	2012-09-05 19:07:35 UTC (rev 127625)
@@ -219,10 +219,11 @@
             CCallHelpers::NotEqual, GPRInfo::nonArgGPR1,
             CCallHelpers::TrustedImm32(JSValue::CellTag)));
 #endif
+    jit.loadPtr(CCallHelpers::Address(GPRInfo::nonArgGPR0, JSCell::structureOffset()), GPRInfo::nonArgGPR2);
     slowCase.append(
         jit.branchPtr(
             CCallHelpers::NotEqual,
-            CCallHelpers::Address(GPRInfo::nonArgGPR0, JSCell::classInfoOffset()),
+            CCallHelpers::Address(GPRInfo::nonArgGPR2, Structure::classInfoOffset()),
             CCallHelpers::TrustedImmPtr(&JSFunction::s_info)));
     
     // Now we know we have a JSFunction.
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to