Closed Bug 943436 Opened 6 years ago Closed 6 years ago

Refactor BacktrackingAllocator::processInterval

Categories

(Core :: JavaScript Engine: JIT, enhancement)

enhancement
Not set

Tracking

()

RESOLVED FIXED
mozilla28

People

(Reporter: sunfish, Assigned: sunfish)

Details

(Whiteboard: [qa-])

Attachments

(1 file)

This patch exacts some code out of BacktrackingAllocator::processInterval into helper functions.

Of particular note is that this changes the fixed-allocation case to just call tryAllocateRegister once, instead of trying every register and having it fail until the fixed requirement is met.
Attachment #8338573 - Flags: review?(bhackett1024)
Attachment #8338573 - Flags: review?(bhackett1024) → review+
https://hg.mozilla.org/mozilla-central/rev/a8dd314fcb9b
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla28
Whiteboard: [qa-]
You need to log in before you can comment on or make changes to this bug.