Closed Bug 677730 Opened 10 years ago Closed 10 years ago

IonMonkey: Snapshots are not assigned to LAdd instructions.

Categories

(Core :: JavaScript Engine, defect)

x86_64
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: adrake, Assigned: adrake)

References

(Blocks 1 open bug)

Details

Attachments

(2 files, 2 obsolete files)

Attached file Test case
The attached test case halts with "0" on ionmonkey tip. Instead, it should bail to the interpreter on overflow and loop forever.
Attached patch Patch v0 (obsolete) — Splinter Review
I also needed to make some small changes to LSRA to ensure the correct value gets captured in the snapshot.
Attachment #551982 - Flags: review?(dvander)
Attached patch Patch v1 (obsolete) — Splinter Review
Now with test case.
Attachment #551982 - Attachment is obsolete: true
Attachment #551982 - Flags: review?(dvander)
Attachment #551983 - Attachment is obsolete: true
Attachment #551985 - Flags: review?(dvander)
Attachment #551985 - Flags: review?(dvander) → review+
http://hg.mozilla.org/projects/ionmonkey/rev/c0262f6e18a4
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.