[Webkit-unassigned] [Bug 19180] speed up SunSpider by optimizing immediate number cases

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Fri May 30 13:12:47 PDT 2008


ap at webkit.org changed:

           What    |Removed                     |Added
  Attachment #21430|review+                     |
               Flag|                            |

------- Comment #28 from ap at webkit.org  2008-05-30 13:12 PDT -------
(From update of attachment 21430)
Clearing review flag, since I landed this.

(In reply to comment #27)
> If you fixed a regression on PowerPC, was there a regression test indicating
> it?

Yes, it's one of the three Mozilla tests that I claimed to have fixed in the
previous patch. Only two were fixed, in fact.

> I'm concerned about the "to" vs. "get" in the name here. This class seems we
> use "get" to mean "convert with no possibility of error" and "to" to mean
> "convert with a possible exception or error".

I'm rather confused about the naming in JSImmediate.h - some "get" functions
also return a bool to indicate success. That said, this new function just
implements "ToUInt32" algorithm from ECMA-232 for immediates as far as I can
tell, and is not really a "get" in the sense that -1 turns into 4294967296. The
"get" functions that we have do not cast the result like that. I wasn't sure
what the word "truncated" meant here, but since this function does almost the
same work as getTruncatedInt32, I kept it.

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

More information about the webkit-unassigned mailing list