Closed Bug 1922499 Opened 1 year ago Closed 1 year ago

Use VirtualRegister::firstRange instead of rangeFor in a few places

Categories

(Core :: JavaScript Engine: JIT, task, P1)

task

Tracking

()

RESOLVED FIXED
133 Branch
Tracking Status
firefox133 --- fixed

People

(Reporter: jandem, Assigned: jandem)

References

Details

Attachments

(1 file)

There are some places where we call VirtualRegister::rangeFor but we just want to get the virtual register's first range. VirtualRegister::rangeFor does a binary search so is also fast, but is still slower than just getting the first range.

This changes some places to use firstRange instead of rangeFor. This is a little
faster but it's also clearer about what the code is doing.

Pushed by jdemooij@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/bee25281b27a Use VirtualRegister::firstRange instead of rangeFor in a few places. r=jseward
Severity: -- → N/A
Priority: -- → P1
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 133 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: