Closed Bug 662082 Opened 9 years ago Closed 9 years ago

TI+JM: some AWFY graphs have a gray background

Categories

(Core :: JavaScript Engine, defect)

defect
Not set

Tracking

()

RESOLVED FIXED

People

(Reporter: jandem, Unassigned)

References

()

Details

Attachments

(1 file)

Some graphs (eg. v8-crypto) on the AWFY breakdown page (http://arewefastyet.com/?a=b&view=breakdown) have a gray background. I can only reproduce this with both TI and JM enabled (default prefs).

I can reproduce with a tinderbox build of revision 53649a1d6e45 on OS X, but I remember seeing it before.
Attached image Screenshot
Did not get a reduction for this, but Jan downloaded the site and identified the place where we were breaking, and with spew output it was not hard to see the problem.  Plain IFEQ and IFNE opcodes did not call fixDoubleTypes before branching, so integers were not being coerced to doubles when branching to a point in the script with associated phi nodes.

http://hg.mozilla.org/projects/jaegermonkey/rev/9ff00d53b5a5
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.