Closed
Bug 714607
Opened 13 years ago
Closed 13 years ago
jit-test/tests/basic/bug680217.js takes a long time
Categories
(Core :: JavaScript Engine, defect)
Core
JavaScript Engine
Tracking
()
RESOLVED
FIXED
mozilla12
People
(Reporter: jandem, Assigned: jandem)
Details
Attachments
(1 file, 1 obsolete file)
|
875 bytes,
patch
|
bhackett1024
:
review+
|
Details | Diff | Splinter Review |
I just got a timeout when running jit-tests. This test takes about 30 seconds to run in the interpreter (with a release build or with -m -n it's much faster)
|
Assignee | |
Comment 1•13 years ago
|
||
Since most changes for bug 680217 were in the parser/emitter, I think it's fine to just lower the iteration count. And even if this value is necessary, I'm sure the fuzzers will catch it again, probably by modifying this testcase.
Attachment #585274 -
Flags: review?(bhackett1024)
|
|
Assignee | |
Comment 2•13 years ago
|
||
Also adds a gc() call (looks like these loops are there just to trigger a GC).
Attachment #585274 -
Attachment is obsolete: true
Attachment #585274 -
Flags: review?(bhackett1024)
Attachment #585275 -
Flags: review?(bhackett1024)
|
||
Updated•13 years ago
|
Attachment #585275 -
Flags: review?(bhackett1024) → review+
|
|
Assignee | |
Comment 3•13 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/06df9b7d4a69
Target Milestone: --- → mozilla12
|
||
Comment 4•13 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/06df9b7d4a69
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•