Closed Bug 1519077 Opened 8 months ago Closed 6 months ago

Int32AddResult and other instructions shouldn't clobber the RHS register

Categories

(Core :: JavaScript Engine: JIT, defect, P1)

defect

Tracking

()

RESOLVED FIXED
mozilla67
Tracking Status
firefox67 --- fixed

People

(Reporter: jandem, Assigned: mgaudet)

References

Details

Attachments

(1 file)

Flags: needinfo?(mgaudet)

(Needinfo for when you get back..)

There's a number of these that I'll fix in one fell swoop.

Assignee: nobody → mgaudet
Status: NEW → ASSIGNED
Flags: needinfo?(mgaudet)

Please be creaful when using AutoScratchRegister outside the MacroAssembler, as this might not work well across all architecture.

Priority: -- → P1
Pushed by mgaudet@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/8a75915b2541
Don't mutate inputs in CacheIR operations r=jandem
Backout by rgurzau@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/c332d5df8ce7
Backed out 2 changesets (bug 1519077, bug 1526024) for jsapi-tests failures on a CLOSED TREE

Backed out 2 changesets (bug 1519077, bug 1526024) for jsapi-tests failures on a CLOSED TREE

Backout link: https://hg.mozilla.org/integration/autoland/rev/c332d5df8ce7cabfcc0224dd319907bfe32780a7

Push with failures: https://treeherder.mozilla.org/#/jobs?repo=autoland&resultStatus=testfailed%2Cbusted%2Cexception&revision=8a75915b2541f5ad01212ec087ea067bf50a1ce0

Failures appeared here: https://treeherder.mozilla.org/#/jobs?repo=autoland&resultStatus=testfailed%2Cbusted%2Cexception&revision=177cf2f812d59db13ad3f7fde4dfa227fe03de21&selectedJob=230628908

Log link: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=230628908&repo=autoland&lineNumber=2073

Log snippet:

[task 2019-02-26T21:40:35.171Z] 21:40:35 INFO - TEST-PASS | testNewContext | ok
[task 2019-02-26T21:40:35.171Z] 21:40:35 INFO - testOOM
[task 2019-02-26T21:40:35.172Z] 21:40:35 INFO - TEST-PASS | testOOM | ok
[task 2019-02-26T21:40:35.172Z] 21:40:35 INFO - testNullRoot
[task 2019-02-26T21:40:35.173Z] 21:40:35 INFO - TEST-PASS | testNullRoot | ok
[task 2019-02-26T21:40:35.173Z] 21:40:35 INFO - testNewTargetInvokeConstructor
[task 2019-02-26T21:40:35.173Z] 21:40:35 INFO - TEST-PASS | testNewTargetInvokeConstructor | ok
[task 2019-02-26T21:40:35.174Z] 21:40:35 INFO - testNewObjrhs clobbered 2(lhsOutput/srcDest) %eax flexibleLshift32 (shift/rhs) %ecx
[task 2019-02-26T21:40:35.174Z] 21:40:35 WARNING - TEST-UNEXPECTED-FAIL | jsapi-tests | test failed with return code -5
[task 2019-02-26T21:40:35.175Z] 21:40:35 INFO - TEST-INFO took 24172ms
[task 2019-02-26T21:40:35.175Z] 21:40:35 INFO - SUITE-END | took 40s
[task 2019-02-26T21:40:35.176Z] 21:40:35 INFO - Result summary:
[task 2019-02-26T21:40:35.176Z] 21:40:35 INFO - cppunittests INFO | Passed: 60
[task 2019-02-26T21:40:35.176Z] 21:40:35 WARNING - cppunittests INFO | Failed: 1
[task 2019-02-26T21:40:35.177Z] 21:40:35 WARNING - One or more unittests failed.
[task 2019-02-26T21:40:35.177Z] 21:40:35 ERROR - Return code: 1
[task 2019-02-26T21:40:35.178Z] 21:40:35 INFO - TinderboxPrint: cppunittest-cppunittest<br/>60/<em class="testfail">1</em>
[task 2019-02-26T21:40:35.178Z] 21:40:35 WARNING - # TBPL FAILURE #
[task 2019-02-26T21:40:35.178Z] 21:40:35 WARNING - setting return code to 2
[task 2019-02-26T21:40:35.178Z] 21:40:35 WARNING - The cppunittest suite: cppunittest ran with return status: FAILURE

Flags: needinfo?(mgaudet)
Pushed by mgaudet@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/46246cd080fa
Don't mutate inputs in CacheIR operations r=jandem
Status: ASSIGNED → RESOLVED
Closed: 6 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla67
You need to log in before you can comment on or make changes to this bug.