Changeset 59282 in webkit for trunk/JavaScriptCore/runtime

Timestamp:

May 12, 2010, 4:25:32 PM (15 years ago)

Author:

[email protected]

Message:

Work around GCC stupidity by either explicitly inlining or by
using ALWAYS_INLINE

Reviewed by Gaving Barraclough.

• runtime/StringPrototype.cpp:

(JSC::stringProtoFuncReplace):

• runtime/UString.h:

(JSC::UString::~UString):

• wtf/RefPtr.h:

(WTF::RefPtr::RefPtr):
(WTF::RefPtr::~RefPtr):

• wtf/text/StringImpl.h:

(WebCore::StringImpl::create):
(WebCore::StringImpl::tryCreateUninitialized):

Location:

trunk/JavaScriptCore/runtime

Files:

• StringPrototype.cpp (modified) (1 diff)
• UString.h (modified) (1 diff)

trunk/JavaScriptCore/runtime/StringPrototype.cpp

@@ -349 +349 @@
                 cachedCall.setThis(exec->globalThisValue());
                 JSValue result = cachedCall.call();
-                replacements.append(result.toString(cachedCall.newCallFrame(exec)));
+                if (LIKELY(result.isString()))
+                    replacements.append(asString(result)->value(exec));
+                else
+                    replacements.append(result.toString(cachedCall.newCallFrame(exec)));
                 if (exec->hadException())
                     break;

trunk/JavaScriptCore/runtime/UString.h

@@ -147 +147 @@
         }
 
+        ALWAYS_INLINE ~UString() { }
     private:
         RefPtr<Rep> m_rep;