Closed
Bug 1057803
Opened 10 years ago
Closed 10 years ago
jit-test/tests/gc/bug-1055034.js will perma-fail when Gecko 34 merges to Aurora
Categories
(Core :: JavaScript: GC, defect)
Core
JavaScript: GC
Tracking
()
VERIFIED
FIXED
mozilla34
Tracking | Status | |
---|---|---|
firefox34 | --- | verified |
People
(Reporter: RyanVM, Assigned: RyanVM)
References
Details
Attachments
(1 file)
(deleted),
patch
|
terrence
:
review+
|
Details | Diff | Splinter Review |
Merge day is Tuesday, 2-Sep, so this is semi-urgent. Presumably just needs an early abort if arr.mapPar isn't supported.
https://tbpl.mozilla.org/php/getParsedLog.php?id=46644860&tree=Try
Assignee | ||
Comment 1•10 years ago
|
||
Looking at the other jit-tests that use mapPar, I guess we'd just want to do this?
-run(range(0, 1024), function (i) { var a = []; a.length = i; });
+if (getBuildConfiguration().parallelJS)
+ run(range(0, 1024), function (i) { var a = []; a.length = i; });
Assignee | ||
Comment 2•10 years ago
|
||
Try says this works.
Comment 3•10 years ago
|
||
Comment on attachment 8478224 [details] [diff] [review]
only run the test if ParallelJS is supported
Review of attachment 8478224 [details] [diff] [review]:
-----------------------------------------------------------------
wfm
Attachment #8478224 -
Flags: review?(terrence) → review+
Assignee | ||
Comment 4•10 years ago
|
||
Assignee | ||
Comment 5•10 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla34
Comment 6•10 years ago
|
||
OK that makes no sense was test only.
Comment 7•10 years ago
|
||
but makes me think bug 1055034 is the regressor.
Assignee | ||
Updated•10 years ago
|
Status: RESOLVED → VERIFIED
status-firefox34:
--- → verified
You need to log in
before you can comment on or make changes to this bug.
Description
•