Changeset 49365 in webkit for trunk/JavaScriptCore/runtime/Collector.h

Timestamp:

Oct 8, 2009, 8:22:41 PM (16 years ago)

Author:

[email protected]

Message:

At long last, I pronounce the death of AllInOneFile.cpp.

Patch by Geoffrey Garen <​[email protected]> on 2009-10-08
Reviewed by Maciej Stachowiak.

SunSpider reports a 1.01x speedup.

• AllInOneFile.cpp: Removed.
• GNUmakefile.am:
• JavaScriptCore.exp:
• JavaScriptCore.gypi:
• JavaScriptCore.xcodeproj/project.pbxproj: Added missing project files

to compilation stages.

• parser/Grammar.y:
• parser/Lexer.cpp:
• parser/Lexer.h:

(JSC::jscyylex):

• runtime/ArrayConstructor.cpp:

(JSC::constructArrayWithSizeQuirk):

• runtime/Collector.h:
• runtime/JSCell.cpp:

(JSC::JSCell::operator new):

• runtime/JSCell.h:

(JSC::JSCell::operator new):

• runtime/JSGlobalObject.cpp:

(JSC::JSGlobalObject::operator new):

• runtime/JSNumberCell.h:

(JSC::JSNumberCell::operator new):

• runtime/JSString.cpp:
• runtime/JSString.h:

(JSC::jsString):
(JSC::jsSubstring):
(JSC::jsOwnedString):

• runtime/RegExpConstructor.cpp:
• runtime/RegExpConstructor.h:

(JSC::RegExpConstructorPrivate::RegExpConstructorPrivate):
(JSC::RegExpConstructorPrivate::lastOvector):
(JSC::RegExpConstructorPrivate::tempOvector):
(JSC::RegExpConstructorPrivate::changeLastOvector):
(JSC::RegExpConstructor::performMatch):

• runtime/StringPrototype.cpp:

(JSC::stringProtoFuncMatch):

• yarr/RegexJIT.cpp:
• yarr/RegexJIT.h:

(JSC::Yarr::executeRegex): Inlined a few things that Shark said
were hot, on the presumption that AllInOneFile.cpp used to inline them
automatically.

File:

• trunk/JavaScriptCore/runtime/Collector.h (modified) (1 diff)

trunk/JavaScriptCore/runtime/Collector.h

@@ -72 +72 @@
         void destroy();
 
-#ifdef JAVASCRIPTCORE_BUILDING_ALL_IN_ONE_FILE
-        // We can inline these functions because everything is compiled as
-        // one file, so the heapAllocate template definitions are available.
-        // However, allocateNumber is used via jsNumberCell outside JavaScriptCore.
-        // Thus allocateNumber needs to provide a non-inline version too.
-        void* inlineAllocateNumber(size_t s) { return heapAllocate<NumberHeap>(s); }
-        void* inlineAllocate(size_t s) { return heapAllocate<PrimaryHeap>(s); }
-#endif
         void* allocateNumber(size_t);
         void* allocate(size_t);