Unreviewed, rolling out r242747.
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 12 Mar 2019 21:03:08 +0000 (21:03 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 12 Mar 2019 21:03:08 +0000 (21:03 +0000)
https://bugs.webkit.org/show_bug.cgi?id=195641

Performance measurement is difficult in this period, rolling
out it and rolling in later to isolate it from the other
sensitive patches (Requested by yusukesuzuki on #webkit).

Reverted changeset:

"[JSC] Make StaticStringImpl & StaticSymbolImpl actually
static"
https://bugs.webkit.org/show_bug.cgi?id=194212
https://trac.webkit.org/changeset/242747

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

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

index 0a3b401..2c77e0e 100644 (file)
@@ -1,3 +1,19 @@
+2019-03-12  Commit Queue  <commit-queue@webkit.org>
+
+        Unreviewed, rolling out r242747.
+        https://bugs.webkit.org/show_bug.cgi?id=195641
+
+        Performance measurement is difficult in this period, rolling
+        out it and rolling in later to isolate it from the other
+        sensitive patches (Requested by yusukesuzuki on #webkit).
+
+        Reverted changeset:
+
+        "[JSC] Make StaticStringImpl & StaticSymbolImpl actually
+        static"
+        https://bugs.webkit.org/show_bug.cgi?id=194212
+        https://trac.webkit.org/changeset/242747
+
 2019-03-12  Robin Morisset  <rmorisset@apple.com>
 
         A lot more classes have padding that can be reduced by reordering their fields
index 293e42d..e564648 100644 (file)
@@ -1064,8 +1064,6 @@ inline void StringImpl::ref()
 {
     STRING_STATS_REF_STRING(*this);
 
-    if (UNLIKELY(isStatic()))
-        return;
     m_refCount += s_refCountIncrement;
 }
 
@@ -1073,8 +1071,6 @@ inline void StringImpl::deref()
 {
     STRING_STATS_DEREF_STRING(*this);
 
-    if (UNLIKELY(isStatic()))
-        return;
     unsigned tempRefCount = m_refCount - s_refCountIncrement;
     if (!tempRefCount) {
         StringImpl::destroy(this);