[Webkit-unassigned] [Bug 64582] JSC JIT does not inline GC allocation fast paths

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=64582





--- Comment #6 from Oliver Hunt <oliver at apple.com>  2011-07-14 21:50:51 PST ---
(In reply to comment #1)
> Created an attachment (id=100924)
 --> (https://bugs.webkit.org/attachment.cgi?id=100924&action=review) [details]
> the patch
> 
> This patch works gives a 1.3% speed-up on V8, and is neutral on SunSpider.
> 
> I've lived on it for a day or so, and it seems to work.  It passes all tests.  But I'm not sure if I'm handling global objects (for initializing the structure in op_new_object) correctly, so I'd appreciate some feedback on that!  Separately I tested the speed-up of op_new_object in isolation, and it wasn't much (statistically significant but fraction of a percent); it definitely appears that op_create_this is doing most of the work.

I'm kind of surprised it doesn't help a microbenchmark of op_new_object, or do you mean only inlining op_new_object doesn't seem to help any of the benchmarks we track? (which is less surprising)

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.