Closed Bug 642324 Opened 9 years ago Closed 9 years ago

TI: Crash on large test case

Categories

(Core :: JavaScript Engine, defect)

x86_64
Linux
defect
Not set

Tracking

()

RESOLVED FIXED

People

(Reporter: dvander, Unassigned)

References

(Blocks 1 open bug, )

Details

Attachments

(1 file)

1.04 KB, application/x-javascript
Details
The test case in bug 591631 doesn't seem to work on the TI branch:

dvander@dvander:~/mozilla/jaegermonkey/js/src$ ./Opt32/js -m fft.js 
fft.js:11461: TypeError: real is undefined
dvander@dvander:~/mozilla/jaegermonkey/js/src$ ./Opt32/js -m -n fft.js 
Segmentation fault
Attached file Reduced
This reduced test case fails with -m -a:

test.js:56: TypeError: spectrum is undefined
This may be related to bug 642569, the test case there also fails with -m -a when using > 50 locals.
This works perfect now with -m and -m -n, I'm sure bug 642569 fixed it.

There are serious FrameState performance issues though, like the ones dvander fixed in bug 591836. Will file a followup bug.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
(In reply to comment #3)
> This works perfect now with -m and -m -n, I'm sure bug 642569 fixed it.
> 
> There are serious FrameState performance issues though, like the ones dvander
> fixed in bug 591836. Will file a followup bug.

These will be addressed in bug 638210
You need to log in before you can comment on or make changes to this bug.