Closed
Bug 634090
Opened 13 years ago
Closed 13 years ago
JIT tests should have a timeout parameter
Categories
(Core :: JavaScript Engine, enhancement)
Core
JavaScript Engine
Tracking
()
RESOLVED
FIXED
People
(Reporter: billm, Assigned: billm)
References
Details
(Whiteboard: fixed-in-tracemonkey)
Attachments
(1 file, 1 obsolete file)
17.23 KB,
patch
|
dmandelin
:
review+
|
Details | Diff | Splinter Review |
This is useful for debugging infinite loops and for testing with GC zeal.
Assignee | ||
Comment 1•13 years ago
|
||
I copied this code from jstests.py. I guess eventually this infrastructure should be shared.
Updated•13 years ago
|
Attachment #512287 -
Flags: review?(dmandelin) → review+
Assignee | ||
Comment 2•13 years ago
|
||
I never felt like this patch was really done. This new one makes it a lot easier to do GC zeal testing. It adds a --args option to jit-tests so you can specify -Z. And for both jit-tests and jstests, it prints timeout failures separate from other kinds of failures. This makes it much easier to see if anything actually went wrong.
Attachment #512287 -
Attachment is obsolete: true
Attachment #527832 -
Flags: review?(dmandelin)
Updated•13 years ago
|
Attachment #527832 -
Flags: review?(dmandelin) → review+
Assignee | ||
Comment 3•13 years ago
|
||
http://hg.mozilla.org/tracemonkey/rev/848855c35e27
Whiteboard: fixed-in-tracemonkey
Comment 4•13 years ago
|
||
cdleary-bot mozilla-central merge info: http://hg.mozilla.org/mozilla-central/rev/848855c35e27
Updated•13 years ago
|
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
•