Remove failing assertion. There are strings that claim to be atomic but that the
authorfpizlo@apple.com <fpizlo@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 9 Mar 2016 07:29:43 +0000 (07:29 +0000)
committerfpizlo@apple.com <fpizlo@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 9 Mar 2016 07:29:43 +0000 (07:29 +0000)
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):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@197838 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WTF/ChangeLog
Source/WTF/wtf/text/StringImpl.h

index 5d3795d..076e155 100644 (file)
@@ -1,5 +1,16 @@
 2016-03-08  Filip Pizlo  <fpizlo@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  <fpizlo@apple.com>
+
         DFG should be able to constant-fold strings
         https://bugs.webkit.org/show_bug.cgi?id=155200
 
index 192db73..4dd924c 100644 (file)
@@ -585,8 +585,6 @@ public:
 
     inline void ref()
     {
-        ASSERT(!isCompilationThread() || !isAtomic());
-
         STRING_STATS_REF_STRING(*this);
 
         m_refCount += s_refCountIncrement;
@@ -594,8 +592,6 @@ public:
 
     inline void deref()
     {
-        ASSERT(!isCompilationThread() || !isAtomic());
-
         STRING_STATS_DEREF_STRING(*this);
 
         unsigned tempRefCount = m_refCount - s_refCountIncrement;