Closed
Bug 718505
Opened 12 years ago
Closed 12 years ago
IonMonkey: LSRA: don't search for blocking intervals if the register was free
Categories
(Core :: JavaScript Engine, defect)
Core
JavaScript Engine
Tracking
()
RESOLVED
FIXED
People
(Reporter: jandem, Assigned: jandem)
References
Details
Attachments
(1 file)
8.96 KB,
patch
|
dvander
:
review+
|
Details | Diff | Splinter Review |
assign() searches active/inactive for blocking intervals, even if findBestFreeRegister found a free register. Simplest fix is to either move the splitting code to a separate function (splitBlockingIntervals or something) or to give assign an optional isFree argument.
Assignee | ||
Comment 1•12 years ago
|
||
Move the splitting code from assign to splitBlockingIntervals, and call it when assigning a blocked register. Applies on top of the patches in bug 712278.
Attachment #589022 -
Flags: review?(dvander)
Updated•12 years ago
|
Attachment #589022 -
Flags: review?(dvander) → review+
Assignee | ||
Comment 2•12 years ago
|
||
https://hg.mozilla.org/projects/ionmonkey/rev/7685f6b2f00d
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•