[Webkit-unassigned] [Bug 26987] Add fast case for Math.floor(a / b) expressions

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Jul 6 06:16:05 PDT 2009


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


Zoltan Herczeg <zherczeg at inf.u-szeged.hu> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #32298|                            |review?
               Flag|                            |




--- Comment #2 from Zoltan Herczeg <zherczeg at inf.u-szeged.hu>  2009-07-06 06:16:04 PDT ---
Created an attachment (id=32298)
 --> (https://bugs.webkit.org/attachment.cgi?id=32298)
Enable math.floor(a/b) optimization on interpreter

This is the main part of the patch. Well, actually it is just a draft, since I
feel you will have several ideas to further improve the code. Furthermore, only
interpreter is supported right now. I would like to hear your opinion before I
continue the work on JIT.

The following code is executed in 0m1.992s without this patch:

var a = 3
for (var i = 0; i < 10000000; ++i)
    Math.floor(a / -2)

The patch improved the runtime to 0m0.790s, which is 2.5 times faster.

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



More information about the webkit-unassigned mailing list