Closed
Bug 1687626
Opened 3 years ago
Closed 3 years ago
Ship Ion for WebAssembly on ARM64
Categories
(Core :: JavaScript: WebAssembly, task, P1)
Tracking
()
RESOLVED
FIXED
90 Branch
Tracking | Status | |
---|---|---|
firefox90 | --- | fixed |
People
(Reporter: lth, Assigned: lth)
References
Details
Attachments
(1 file)
Prototyping work (bug 1678097) has demonstrated that Ion can generate good code quickly for wasm on ARM64, and given that Ion has good stability and we know it well, we will ship it as the initial optimizing compiler for wasm on that platform.
Assignee | ||
Updated•3 years ago
|
Assignee | ||
Comment 1•3 years ago
|
||
When we land this, we will land in two phases:
Phase 1:
- All base SIMD patches (bug 1678097, bug 1687629)
- A patch that allows Ion to be used on arm64 by a shell command line switch (maybe --wasm-compiler=ion, maybe something else), but which leaves cranelift enabled and on by default
- Probably a patch to request ion in Firefox nightly builds by flipping a flag in about:config
When Phase 1 is complete, fuzzing can use the new command line switch, but we are not at risk of exposing users to fresh code and we are not inviting spurious bug bounties ditto.
Phase 2:
- A patch that disables cranelift (but leaves it a configure-time option, which if selected will disable ion) and removes the command line switch and any about:config option introduced in Phase 1.
When Phase 2 is complete, Ion is live in FF Nightly builds on ARM64.
Assignee | ||
Updated•3 years ago
|
Assignee: lhansen → nobody
Status: ASSIGNED → NEW
Assignee | ||
Updated•3 years ago
|
Assignee: nobody → jseward
Status: NEW → ASSIGNED
Assignee | ||
Updated•3 years ago
|
Depends on: CVE-2021-29981
Assignee | ||
Updated•3 years ago
|
Priority: P2 → P1
Assignee | ||
Updated•3 years ago
|
Assignee | ||
Updated•3 years ago
|
No longer depends on: CVE-2021-29981
Assignee | ||
Updated•3 years ago
|
Assignee: jseward → lhansen
Assignee | ||
Comment 2•3 years ago
|
||
We are feature complete and there are no blocking bugs,
let it ride the trains.
Pushed by lhansen@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/6929aab7f34a Enable Ion+SIMD on ARM64 on all channels. r=jseward
Comment 4•3 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
status-firefox90:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 90 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•