Title: [197838] trunk/Source/WTF
Revision
197838
Author
fpi...@apple.com
Date
2016-03-08 23:29:43 -0800 (Tue, 08 Mar 2016)

Log Message

Remove failing assertion. There are strings that claim to be atomic but that the
compiler thread can totally deal with, like the empty string.

Rubber stamped by Mark Lam.

* wtf/text/StringImpl.h:
(WTF::StringImpl::ref):
(WTF::StringImpl::deref):

Modified Paths

Diff

Modified: trunk/Source/WTF/ChangeLog (197837 => 197838)


--- trunk/Source/WTF/ChangeLog	2016-03-09 07:28:58 UTC (rev 197837)
+++ trunk/Source/WTF/ChangeLog	2016-03-09 07:29:43 UTC (rev 197838)
@@ -1,5 +1,16 @@
 2016-03-08  Filip Pizlo  <fpi...@apple.com>
 
+        Remove failing assertion. There are strings that claim to be atomic but that the
+        compiler thread can totally deal with, like the empty string.
+
+        Rubber stamped by Mark Lam.
+
+        * wtf/text/StringImpl.h:
+        (WTF::StringImpl::ref):
+        (WTF::StringImpl::deref):
+
+2016-03-08  Filip Pizlo  <fpi...@apple.com>
+
         DFG should be able to constant-fold strings
         https://bugs.webkit.org/show_bug.cgi?id=155200
 

Modified: trunk/Source/WTF/wtf/text/StringImpl.h (197837 => 197838)


--- trunk/Source/WTF/wtf/text/StringImpl.h	2016-03-09 07:28:58 UTC (rev 197837)
+++ trunk/Source/WTF/wtf/text/StringImpl.h	2016-03-09 07:29:43 UTC (rev 197838)
@@ -585,8 +585,6 @@
 
     inline void ref()
     {
-        ASSERT(!isCompilationThread() || !isAtomic());
-
         STRING_STATS_REF_STRING(*this);
 
         m_refCount += s_refCountIncrement;
@@ -594,8 +592,6 @@
 
     inline void deref()
     {
-        ASSERT(!isCompilationThread() || !isAtomic());
-
         STRING_STATS_DEREF_STRING(*this);
 
         unsigned tempRefCount = m_refCount - s_refCountIncrement;
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to