Title: [199384] trunk/Source/_javascript_Core
Revision
199384
Author
sbar...@apple.com
Date
2016-04-12 13:47:24 -0700 (Tue, 12 Apr 2016)

Log Message

isLocked() assertion broke builds because ConcurrentJITLock isn't always a real lock.

Rubber-stamped by Filip Pizlo.

* bytecode/CodeBlock.cpp:
(JSC::CodeBlock::resultProfileForBytecodeOffset):
(JSC::CodeBlock::ensureResultProfile):

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (199383 => 199384)


--- trunk/Source/_javascript_Core/ChangeLog	2016-04-12 20:40:41 UTC (rev 199383)
+++ trunk/Source/_javascript_Core/ChangeLog	2016-04-12 20:47:24 UTC (rev 199384)
@@ -1,3 +1,13 @@
+2016-04-12  Saam barati  <sbar...@apple.com>
+
+        isLocked() assertion broke builds because ConcurrentJITLock isn't always a real lock.
+
+        Rubber-stamped by Filip Pizlo.
+
+        * bytecode/CodeBlock.cpp:
+        (JSC::CodeBlock::resultProfileForBytecodeOffset):
+        (JSC::CodeBlock::ensureResultProfile):
+
 2016-04-11  Filip Pizlo  <fpi...@apple.com>
 
         PolymorphicAccess should buffer AccessCases before regenerating

Modified: trunk/Source/_javascript_Core/bytecode/CodeBlock.cpp (199383 => 199384)


--- trunk/Source/_javascript_Core/bytecode/CodeBlock.cpp	2016-04-12 20:40:41 UTC (rev 199383)
+++ trunk/Source/_javascript_Core/bytecode/CodeBlock.cpp	2016-04-12 20:47:24 UTC (rev 199384)
@@ -4235,7 +4235,6 @@
 
 ResultProfile* CodeBlock::resultProfileForBytecodeOffset(const ConcurrentJITLocker&, int bytecodeOffset)
 {
-    ASSERT(m_lock.isLocked());
     if (!m_bytecodeOffsetToResultProfileIndexMap)
         return nullptr;
     auto iterator = m_bytecodeOffsetToResultProfileIndexMap->find(bytecodeOffset);
@@ -4253,7 +4252,6 @@
 
 ResultProfile* CodeBlock::ensureResultProfile(const ConcurrentJITLocker& locker, int bytecodeOffset)
 {
-    ASSERT(m_lock.isLocked());
     ResultProfile* profile = "" bytecodeOffset);
     if (!profile) {
         m_resultProfiles.append(ResultProfile(bytecodeOffset));
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to