469370 - TM: foreach loop with sarr.length 3-4x slower with jit

Status: RESOLVED INVALID

(Core :: JavaScript Engine, defect)

Description

[Dorus Peelen]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 6.0; nl; rv:1.9.0.4) Gecko/2008102920 Firefox/3.0.4
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9.2a1pre) Gecko/20081212 Minefield/3.2a1pre

as mentioned in bug 451974 comment #27, this is fixed in the tracemonkey builds (10-20% faster), but didnt make it into the recent merge.

code:

var f=function(x){}; for (var i=0, len=sarr.length; i<len; i++) { f(sarr[i]); }

Reproducible: Always

Comment 1

[Dorus Peelen]

Odd how i'm suddenly unable to reproduce this. I'll take a close look when i got more time.

Comment 2

[Dorus Peelen]

Oh well, i still got the benchmark data where it this goes wrong, so i didn't imagine it, but i can't find a single build back where this test goes wrong, and the other for each not. Guess this was just a copy-paste error on my side or so.