Title: [129428] trunk/Source/_javascript_Core
Revision
129428
Author
[email protected]
Date
2012-09-24 16:53:11 -0700 (Mon, 24 Sep 2012)

Log Message

Added a missing "if VALUE_PROFILER" around an access to ArrayProfile record.
https://bugs.webkit.org/show_bug.cgi?id=97496.

Reviewed by Filip Pizlo.

* llint/LowLevelInterpreter32_64.asm:
* llint/LowLevelInterpreter64.asm:

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (129427 => 129428)


--- trunk/Source/_javascript_Core/ChangeLog	2012-09-24 23:43:28 UTC (rev 129427)
+++ trunk/Source/_javascript_Core/ChangeLog	2012-09-24 23:53:11 UTC (rev 129428)
@@ -1,3 +1,13 @@
+2012-09-24  Mark Lam  <[email protected]>
+
+        Added a missing "if VALUE_PROFILER" around an access to ArrayProfile record.
+        https://bugs.webkit.org/show_bug.cgi?id=97496.
+
+        Reviewed by Filip Pizlo.
+
+        * llint/LowLevelInterpreter32_64.asm:
+        * llint/LowLevelInterpreter64.asm:
+
 2012-09-24  Geoffrey Garen  <[email protected]>
 
         Inlined activation tear-off in the DFG

Modified: trunk/Source/_javascript_Core/llint/LowLevelInterpreter32_64.asm (129427 => 129428)


--- trunk/Source/_javascript_Core/llint/LowLevelInterpreter32_64.asm	2012-09-24 23:43:28 UTC (rev 129427)
+++ trunk/Source/_javascript_Core/llint/LowLevelInterpreter32_64.asm	2012-09-24 23:53:11 UTC (rev 129428)
@@ -1398,7 +1398,9 @@
     dispatch(5)
 
 .opPutByValEmpty:
-    storeb 1, ArrayProfile::m_mayStoreToHole[t3]
+    if VALUE_PROFILER
+        storeb 1, ArrayProfile::m_mayStoreToHole[t3]
+    end
     addi 1, ArrayStorage::m_numValuesInVector[t0]
     bib t2, -sizeof IndexingHeader + IndexingHeader::m_publicLength[t0], .opPutByValStoreResult
     addi 1, t2, t1

Modified: trunk/Source/_javascript_Core/llint/LowLevelInterpreter64.asm (129427 => 129428)


--- trunk/Source/_javascript_Core/llint/LowLevelInterpreter64.asm	2012-09-24 23:43:28 UTC (rev 129427)
+++ trunk/Source/_javascript_Core/llint/LowLevelInterpreter64.asm	2012-09-24 23:53:11 UTC (rev 129428)
@@ -1241,7 +1241,9 @@
     dispatch(5)
 
 .opPutByValEmpty:
-    storeb 1, ArrayProfile::m_mayStoreToHole[t3]
+    if VALUE_PROFILER
+        storeb 1, ArrayProfile::m_mayStoreToHole[t3]
+    end
     addi 1, ArrayStorage::m_numValuesInVector[t0]
     bib t2, -sizeof IndexingHeader + IndexingHeader::m_publicLength[t0], .opPutByValStoreResult
     addi 1, t2, t1
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to