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)
Core
JavaScript Engine: JIT
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.
| Assignee | ||
Comment 1•1 year ago
|
||
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
Updated•1 year ago
|
Severity: -- → N/A
Priority: -- → P1
Comment 3•1 year ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
status-firefox133:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 133 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•