Ignore:
Timestamp:
May 11, 2010, 7:25:40 PM (15 years ago)
Author:
[email protected]
Message:

Patch by Gavin Barraclough.

Reviewed by Maciej Stachowiak.

Rest of REGRESSION (r57900-57919): 3% PLT Regression from moving strings into WTF.
https://bugs.webkit.org/show_bug.cgi?id=38930
<rdar://problem/7937188>

Inline string functions that are commonly used in WTFString, CString, and AtomicString.

  • JavaScriptCore.exp: Remove exports of inline functions.
  • wtf/text/AtomicString.cpp:

(WebCore::AtomicString::addSlowCase): Rename add to addSlowCase, and inline the fast path

of add.

  • wtf/text/AtomicString.h:

(WebCore::AtomicString::add): Inline the fast path of add.

  • wtf/text/CString.cpp: Inline implementation of data and length in the header.
  • wtf/text/CString.h:

(WTF::CString::data): Inline implementation of data.
(WTF::CString::length): Inline implementation of length.

  • wtf/text/WTFString.cpp: Remove implementations of functions to be inlined.
  • wtf/text/WTFString.h:

(WebCore::String::String): Inline implementation.
(WebCore::String::length): Ditto.
(WebCore::String::characters): Ditto.
(WebCore::String::operator[]): Ditto.
(WebCore::String::isEmpty): Ditto.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/JavaScriptCore/JavaScriptCore.exp

    r59067 r59196  
     1__ZN7WebCore10StringImpl6createEPKcj
     2__ZN7WebCore12AtomicString11addSlowCaseEPNS_10StringImplE
    13_JSCheckScriptSyntax
    24_JSClassCreate
     
    406408__ZN7WebCore12AtomicString3addEPKtj
    407409__ZN7WebCore12AtomicString3addEPKtjj
    408 __ZN7WebCore12AtomicString3addEPNS_10StringImplE
    409410__ZN7WebCore12AtomicString4findEPKtjj
    410411__ZN7WebCore12AtomicString4initEv
     
    440441__ZN7WebCore6String8fromUTF8EPKcm
    441442__ZN7WebCore6String8truncateEj
    442 __ZN7WebCore6StringC1EPKc
    443 __ZN7WebCore6StringC1EPKcj
    444443__ZN7WebCore6StringC1EPKt
    445 __ZN7WebCore6StringC1EPKtj
    446444__ZN7WebCore7xmlAtomE
    447445__ZN7WebCore8nullAtomE
     
    511509__ZNK3JSC9HashTable11createTableEPNS_12JSGlobalDataE
    512510__ZNK3JSC9HashTable11deleteTableEv
    513 __ZNK3WTF7CString4dataEv
    514 __ZNK3WTF7CString6lengthEv
    515511__ZNK3WTF8Collator7collateEPKtmS2_m
    516512__ZNK7WebCore12AtomicString5lowerEv
    517 __ZNK7WebCore6String10charactersEv
    518513__ZNK7WebCore6String11toIntStrictEPbi
    519514__ZNK7WebCore6String12toUIntStrictEPbi
     
    534529__ZNK7WebCore6String5upperEv
    535530__ZNK7WebCore6String6latin1Ev
    536 __ZNK7WebCore6String6lengthEv
    537531__ZNK7WebCore6String6toUIntEPb
    538 __ZNK7WebCore6String7isEmptyEv
    539532__ZNK7WebCore6String7toFloatEPb
    540533__ZNK7WebCore6String8foldCaseEv
     
    543536__ZNK7WebCore6String8toUInt64EPb
    544537__ZNK7WebCore6String9substringEjj
    545 __ZNK7WebCore6StringixEj
    546538__ZTVN3JSC12StringObjectE
    547539__ZTVN3JSC14JSGlobalObjectE
Note: See TracChangeset for help on using the changeset viewer.