Closed
Bug 1726122
Opened 3 years ago
Closed 3 years ago
Move functions from MIR.h into caller file
Categories
(Core :: JavaScript Engine: JIT, task, P1)
Core
JavaScript Engine: JIT
Tracking
()
RESOLVED
FIXED
93 Branch
| Tracking | Status | |
|---|---|---|
| firefox93 | --- | fixed |
People
(Reporter: anba, Assigned: anba)
Details
Attachments
(2 files)
MIR.h has some functions which are only called within a single cpp file, so we can actually move these functions into the cpp files. And there's also an AlignmentFinder specialisation which can be removed after bug 1448387.
|
Assignee | |
Comment 1•3 years ago
|
||
These specialisations are no longer needed after bug 1448387.
|
|
Assignee | |
Comment 2•3 years ago
|
||
Drive-by change:
Rename MIRTypeForArrayBufferViewRead parameter to forceDoubleForUint32 to
match the argument name in the callers.
Depends on D122837
Pushed by andre.bargull@gmail.com: https://hg.mozilla.org/integration/autoland/rev/fe00a06e58ac Part 1: Remove AlignmentFinder specialisations from MIR.h. r=jandem https://hg.mozilla.org/integration/autoland/rev/d6e1ffa00372 Part 2: Move MIR.h functions with single caller. r=jandem,nbp
|
||
Updated•3 years ago
|
Severity: -- → N/A
Priority: -- → P1
|
||
Comment 4•3 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/fe00a06e58ac
https://hg.mozilla.org/mozilla-central/rev/d6e1ffa00372
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 93 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•