Changeset 34378 in webkit for trunk/JavaScriptCore

Timestamp:

Jun 5, 2008, 10:54:59 AM (17 years ago)

Author:

[email protected]

Message:

2008-06-05 Maciej Stachowiak <​[email protected]>

Reviewed by Hyatt.

• force inlining of a template function that only has one call site per specialization 1.3% speedup on SunSpider

• kjs/collector.cpp: (KJS::Collector::heapAllocate): This template function is only called from allocate() and allocateNumber() (once per specialization) and the extra call overhead for GC allocation shows up, so force inlining.

Location:

trunk/JavaScriptCore

Files:

• ChangeLog (modified) (1 diff)
• kjs/collector.cpp (modified) (1 diff)

trunk/JavaScriptCore/ChangeLog

@@ +1 @@
+2008-06-05  Maciej Stachowiak  <[email protected]>
+
+        Reviewed by Hyatt.
+        
+        - force inlining of a template function that only has one call site per specialization
+        1.3% speedup on SunSpider
+
+        * kjs/collector.cpp:
+        (KJS::Collector::heapAllocate): This template function is only
+        called from allocate() and allocateNumber() (once per
+        specialization) and the extra call overhead for GC allocation
+        shows up, so force inlining.
+
 2008-06-05  Maciej Stachowiak  <[email protected]>
 

trunk/JavaScriptCore/kjs/collector.cpp

@@ -179 +179 @@
 };
 
-template <Collector::HeapType heapType> void* Collector::heapAllocate(size_t s)
+template <Collector::HeapType heapType> ALWAYS_INLINE void* Collector::heapAllocate(size_t s)
 {
     typedef typename HeapConstants<heapType>::Block Block;