[webkit-reviews] review granted: [Bug 113862] Negative zero checks cause unnecessary speculation failures on SunSpider on ARMv7 : [Attachment 197263] New patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Apr 25 12:34:47 PDT 2013


Geoffrey Garen <ggaren at apple.com> has granted Roman Zhuykov
<zhroma at ispras.ru>'s request for review:
Bug 113862: Negative zero checks cause unnecessary speculation failures on
SunSpider on ARMv7
https://bugs.webkit.org/show_bug.cgi?id=113862

Attachment 197263: New patch
https://bugs.webkit.org/attachment.cgi?id=197263&action=review

------- Additional Comments from Geoffrey Garen <ggaren at apple.com>
View in context: https://bugs.webkit.org/attachment.cgi?id=197263&action=review


r=me

cq- because of the layout test issue. Any committer can fix the layout test and
then land, or you can post a new patch, and I can cq+ it.

> LayoutTests/fast/js/regress/script-tests/negative-zero-divide.js:6
> +  for (i = 0; i < 100000; i++) {

Please change this to 100. That's our de facto limit for getting code to run in
the DFG, without making regression tests run too long.

> LayoutTests/fast/js/regress/script-tests/negative-zero-divide.js:7
> +    if (i == 99999)

And 99 here.

> LayoutTests/fast/js/regress/script-tests/negative-zero-modulo.js:6
> +  for (i = 0; i < 100000; i++) {
> +    if (i == 99999)

Ditto.

> LayoutTests/fast/js/regress/script-tests/negative-zero-negate.js:6
> +  for (i = 0; i < 100000; i++) {
> +    if (i == 99999)

Ditto.


More information about the webkit-reviews mailing list