Closed
Bug 998262
Opened 10 years ago
Closed 10 years ago
Differential Testing: Different output message involving buildPar
Categories
(Core :: JavaScript Engine: JIT, defect)
Core
JavaScript Engine: JIT
Tracking
()
RESOLVED
FIXED
mozilla32
People
(Reporter: gkw, Assigned: shu)
Details
(Keywords: regression, testcase)
Attachments
(1 file)
1.41 KB,
patch
|
nmatsakis
:
review+
|
Details | Diff | Splinter Review |
print( Array.buildPar(3, function() { return [] !== 0.1 }) ) $ ./js-opt-64-dm-ts-darwin-7fe3ee0cf8be --fuzzing-safe --ion-parallel-compile=off testcase.js true,true,false $ ./js-opt-64-dm-ts-darwin-7fe3ee0cf8be --fuzzing-safe --ion-parallel-compile=off --no-baseline testcase.js true,true,true (Tested this on 64-bit Mac js opt threadsafe deterministic shell off m-c rev 7fe3ee0cf8be, and I think it also reproduces on Linux) My configure flags (Mac) are: CC="clang -Qunused-arguments" CXX="clang++ -Qunused-arguments" AR=ar sh /Users/skywalker/trees/mozilla-central/js/src/configure --target=x86_64-apple-darwin12.5.0 --enable-optimize --disable-debug --enable-profiling --enable-gczeal --enable-debug-symbols --disable-tests --enable-more-deterministic --with-ccache --enable-threadsafe <other NSPR options> This blocks differential testing fuzzing with buildPar.
Reporter | ||
Comment 1•10 years ago
|
||
autoBisect shows this is probably related to the following changeset: The first bad revision is: changeset: http://hg.mozilla.org/mozilla-central/rev/ad7777f1c0f5 user: Shu-yu Guo date: Fri Feb 07 14:40:29 2014 -0800 summary: Bug 958370 - Unify chunks and slices; self-host the scheduler's slice processing loop. (r=nmatsakis) Shu-yu, is bug 958370 a likely regressor?
Flags: needinfo?(shu)
Assignee | ||
Comment 2•10 years ago
|
||
Attachment #8425828 -
Flags: review?(nmatsakis)
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → shu
Status: NEW → ASSIGNED
Flags: needinfo?(shu)
Updated•10 years ago
|
Attachment #8425828 -
Flags: review?(nmatsakis) → review+
Reporter | ||
Comment 3•10 years ago
|
||
Thanks for fixing this! Is this ready for landing?
Flags: needinfo?(shu)
Assignee | ||
Comment 4•10 years ago
|
||
(In reply to Gary Kwong [:gkw] [:nth10sd] from comment #3) > Thanks for fixing this! Is this ready for landing? Why yes: https://hg.mozilla.org/integration/mozilla-inbound/rev/9fc8626c0cb7
Flags: needinfo?(shu)
Comment 5•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/9fc8626c0cb7
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla32
You need to log in
before you can comment on or make changes to this bug.
Description
•