[webkit-reviews] review requested: [Bug 26987] Add fast case for Math.floor(a / b) expressions : [Attachment 32298] Enable math.floor(a/b) optimization on interpreter

[bugzilla-daemon at webkit.org]

Zoltan Herczeg <zherczeg at inf.u-szeged.hu> has asked  for review:
Bug 26987: Add fast case for Math.floor(a / b) expressions
https://bugs.webkit.org/show_bug.cgi?id=26987

Attachment 32298: Enable math.floor(a/b) optimization on interpreter
https://bugs.webkit.org/attachment.cgi?id=32298&action=edit

------- Additional Comments from Zoltan Herczeg <zherczeg at inf.u-szeged.hu>
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.