Closed
Bug 1342328
Opened 7 years ago
Closed 7 years ago
Intermittent tests/jit-test/jit-test/tests/ion/dce-with-rinstructions.js | Unknown (code -11, args "")
Categories
(Core :: JavaScript Engine: JIT, defect)
Core
JavaScript Engine: JIT
Tracking
()
RESOLVED
FIXED
mozilla55
Tracking | Status | |
---|---|---|
firefox-esr52 | --- | unaffected |
firefox53 | --- | unaffected |
firefox54 | --- | fixed |
firefox55 | --- | fixed |
People
(Reporter: intermittent-bug-filer, Assigned: nbp)
Details
(Keywords: intermittent-failure)
Attachments
(1 file)
1.99 KB,
patch
|
tcampbell
:
review+
|
Details | Diff | Splinter Review |
Filed by: philringnalda [at] gmail.com https://treeherder.mozilla.org/logviewer.html#?job_id=79822870&repo=mozilla-central https://archive.mozilla.org/pub/firefox/tinderbox-builds/mozilla-central-macosx64/1487896056/mozilla-central_yosemite_r7_test-jittest-bm134-tests1-macosx-build11.txt.gz
Comment hidden (Intermittent Failures Robot) |
Assignee | ||
Updated•7 years ago
|
Flags: needinfo?(nicolas.b.pierron)
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment 4•7 years ago
|
||
this is osx opt only, not a high failure rate, but creeping up, 20 times in the last week, no easy sign to see of where the increase started- could just be random luck
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Comment hidden (Intermittent Failures Robot) |
Assignee | ||
Comment 12•7 years ago
|
||
I am able to reproduce a similar issue while running on Linux with "rr record -h".
Assignee: nobody → nicolas.b.pierron
Status: NEW → ASSIGNED
Assignee | ||
Comment 13•7 years ago
|
||
These test cases are supposed to work as follow: - They run a few number of times with a single variant of the same function, and inline the only known variant of it, which contains "return false;" - Then Unreachable Code Elimination removes of the testing branch, and the usage of the variables from it. - The function assertRecoveredOnBailout then asserts that the first input is recovered as expected, when it is being encoded. The problem was that by compiling the top-level for-loop, we prevented the inlining of the uceFault* functions, which thus prevent the removal ofthe testing branches, and the removal of the uses, thus the failure of the assertRecoveredOnBailout functions as recover instructions. This patch adds "with({}){}" no-op statement in the top-level for loop, which has the side-effect of preventing the compilation of the for loop with IonMonkey.
Attachment #8866414 -
Flags: review?(tcampbell)
Assignee | ||
Updated•7 years ago
|
Flags: needinfo?(nicolas.b.pierron)
Comment 14•7 years ago
|
||
Comment on attachment 8866414 [details] [diff] [review] IonMonkey dce-with-rinstructions.js test: Prevent compilation of the top-level loop. Review of attachment 8866414 [details] [diff] [review]: ----------------------------------------------------------------- In the future it would be nice to have something more robust than |with({}){}|. Otherwise, looks good.
Attachment #8866414 -
Flags: review?(tcampbell) → review+
Comment 15•7 years ago
|
||
Pushed by npierron@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/b73d9e993d96 IonMonkey dce-with-rinstructions.js test: Prevent compilation of the top-level loop. r=tcampbell
Comment 16•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/b73d9e993d96
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
status-firefox55:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
Updated•7 years ago
|
status-firefox53:
--- → unaffected
status-firefox54:
--- → affected
status-firefox-esr52:
--- → unaffected
Comment 17•7 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-beta/rev/d379fb6970a2
Flags: in-testsuite+
You need to log in
before you can comment on or make changes to this bug.
Description
•