Closed Bug 1687626 Opened 3 years ago Closed 3 years ago

Ship Ion for WebAssembly on ARM64

Categories

(Core :: JavaScript: WebAssembly, task, P1)

ARM64
All
task

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.

Depends on: 1686626, 1678097
Depends on: 1687629
Depends on: 1687630

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: lhansen → nobody
Status: ASSIGNED → NEW
Assignee: nobody → jseward
Status: NEW → ASSIGNED
Blocks: 1695589
Depends on: 1697560
Depends on: CVE-2021-29981
Priority: P2 → P1
Depends on: 1709803
No longer depends on: 1687630
No longer blocks: 1695589
Depends on: 1695599
Depends on: 1710312
Depends on: 1709772
No longer depends on: 1695599
Depends on: 1711636
No longer depends on: CVE-2021-29981
Assignee: jseward → lhansen

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
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 90 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: