Closed
Bug 772830
Opened 13 years ago
Closed 10 years ago
IonMonkey: Regalloc can assign the same register to multiple operands with useRegisterAtStart
Categories
(Core :: JavaScript Engine, defect)
Core
JavaScript Engine
Tracking
()
RESOLVED
WONTFIX
People
(Reporter: jandem, Assigned: jandem)
References
Details
(Whiteboard: [ion:t][leave open])
Attachments
(2 files)
|
8.15 KB,
patch
|
Details | Diff | Splinter Review | |
|
3.91 KB,
patch
|
dvander
:
review+
|
Details | Diff | Splinter Review |
No description provided.
|
Assignee | |
Comment 1•13 years ago
|
||
Christian, as we discussed on IRC, can you fuzz this patch for a while on IonMonkey revision bf456ba39108 (later revisions are orange)? x86-only, I added some (temporary) hacks to exercise the code better but these changes break x64.
Attachment #641041 -
Flags: feedback?(choller)
|
||
Updated•13 years ago
|
Whiteboard: [ion:t]
|
||
Comment 2•13 years ago
|
||
What's the current state of this bug, does it still need fuzzing? If so, can we get a patch rebased for m-c?
|
|
Assignee | |
Updated•13 years ago
|
Attachment #641041 -
Flags: feedback?(choller)
|
|
Assignee | |
Comment 3•13 years ago
|
||
Unfortunately I don't have the time to look into this atm. To support this case we have to review all places where we split intervals etc. For now I think the right thing to do is restrict its use so that it doesn't bite people in the meantime.
Attachment #682398 -
Flags: review?(dvander)
|
|
||
Updated•13 years ago
|
Attachment #682398 -
Flags: review?(dvander) → review+
|
|
Assignee | |
Comment 4•13 years ago
|
||
Whiteboard: [ion:t] → [ion:t][leave open]
|
||
Comment 5•13 years ago
|
||
|
|
Assignee | |
Comment 6•10 years ago
|
||
LSRA is now disabled by default (and will be removed soon) so we can close this.
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•