Closed Bug 1668398 Opened 4 years ago Closed 4 years ago

Vendor Cranelift to get Wasm ABI-2020 support

Categories

(Core :: JavaScript: WebAssembly, enhancement)

enhancement

Tracking

()

RESOLVED FIXED
83 Branch
Tracking Status
firefox83 --- fixed

People

(Reporter: cfallin, Assigned: cfallin)

References

Details

Attachments

(1 file)

In order to support the new Wasm ABI when using a Cranelift-based backend (bug 1657802), we need to pull in a version of Cranelift that includes PR #2223. We should be able to vendor this independently of the rest of the ABI patches, because the Cranelift-side support exists as a new ABI option.

Blocks: 1657802

This patch pulls in the latest version of Cranelift, which includes
necessary updates to support some recent work on the Wasm backend (e.g.,
support for the new ABI in PR #2223).

Pushed by cfallin@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/92a01ad5e890 vendor Cranelift b8f0dc429f2b886e7423122223393b2c7ee3cd4f. r=bbouvier

Hmm. This failure is due to some bitrot in an experimental new ARM32 backend in Cranelift; it appears that on the Cranelift side, the arm32 build-config feature means "new experimental backend", while on the SpiderMonkey side, we pass arm32 just to mean "hey we're on an ARM32 machine". Unfortunate overloading of this feature name loads to an inadvertent build dependency (although it appears we'd never actually invoke the Cranelift backend).

On the Cranelift side, I have a fix for the bitrot here (#2259) and I'll look into adding a build-only test to our CL/wasmtime CI. On the SpiderMonkey side, I think we should exclude Cranelift altogether on platforms that are not explicitly x86-64 or aarch64. I'll create a separate patch for that.

Flags: needinfo?(cfallin)
Depends on: 1668910
Attachment #9178863 - Attachment description: Bug 1668398: vendor Cranelift b8f0dc429f2b886e7423122223393b2c7ee3cd4f. r=bbouvier → Bug 1668398: vendor Cranelift 57fed697920cb888c6cb7e406d13518f7edd12ea. r=bbouvier
Pushed by cfallin@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/d548bb7ebca0 vendor Cranelift 57fed697920cb888c6cb7e406d13518f7edd12ea. r=bbouvier
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 83 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: