Closed Bug 900357 Opened 6 years ago Closed 6 years ago

IonMonkey: (ARM) Long Standing scratchregister/secondscratchregister mismatch

Categories

(Core :: JavaScript Engine, defect)

ARM
Linux
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla25

People

(Reporter: mjrosenb, Unassigned)

Details

Attachments

(1 file)

Somehow or other, this incorrect use of ScratchRegister has remained.  It has escaped all testing, only showing itself on awfy *with* ionparallel compilation.
Attachment #784245 - Flags: review?(Jacob.Bramley)
Comment on attachment 784245 [details] [diff] [review]
/home/mjrosenb/patches/badScratchRegister-r0.patch

Review of attachment 784245 [details] [diff] [review]:
-----------------------------------------------------------------

Do you know why it only shows up with parallel compilation? Does that require larger offsets for some reason?
Attachment #784245 - Flags: review?(Jacob.Bramley) → review+
https://hg.mozilla.org/mozilla-central/rev/db6598c7675f
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla25
You need to log in before you can comment on or make changes to this bug.