Summary: | Add float/double specific versions of getUInt32() for a 0.5% speedup in SunSpider | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Eric Seidel (no email) <eric> | ||||||
Component: | JavaScriptCore | Assignee: | Eric Seidel (no email) <eric> | ||||||
Status: | RESOLVED WONTFIX | ||||||||
Severity: | Normal | CC: | darin, oliver | ||||||
Priority: | P2 | ||||||||
Version: | 523.x (Safari 3) | ||||||||
Hardware: | Mac | ||||||||
OS: | OS X 10.4 | ||||||||
Attachments: |
|
Description
Eric Seidel (no email)
2007-10-22 00:41:52 PDT
Created attachment 16786 [details]
patch
Holy crap! I just realized this was a 68% speedup for nseive-bits! wooo hooo! Bleh. I'm not even sure this is worth it anymore. Turns out Shark was sending me to the wrong source file. I've learned my lesson. This might still be worth landing. Created attachment 16789 [details]
final patch
So the major speedup before was due to a bug in the initial patch. That bug is now fixed, and this turns out to be a much smaller speedup. I actually show more time being spent in this function (under shark). In bug 15617 I do this same optimization and a few others. We should probably take mine. Oliver found another way to solve this. Closing. |