1668910 - Include Cranelift backend only for explicitly-acceptable architectures: x86-64 and arm64.

Status: RESOLVED FIXED

(Core :: JavaScript: WebAssembly, defect)

Description

[Chris Fallin [:cfallin]]

In #1668398, we discovered that Cranelift is built even on architectures where it has no functioning backend. In that particular case, the neutral-sounding Cargo feature arm32 actually enabled an experimental, incomplete ARM32 backend that failed to build. We should avoid including Cranelift in the build unless we're on a platform with a known-working, fully-supported backend.

Comment 1

[Chris Fallin [:cfallin]]

Attachment: Bug 1668910: don't enable Cranelift by default unless target is ARM64 or x86-64. r=lth

These two platforms are the only ones where Cranelift has a functioning
backend. It doesn't make sense to include Cranelift in the build
otherwise, and doing so may cause other issues or breakage.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/fe06bfb5e8c6
don't enable Cranelift by default unless target is ARM64 or x86-64. r=lth

Comment 3

[Narcis Beleuzu [:NarcisB]]

https://hg.mozilla.org/mozilla-central/rev/fe06bfb5e8c6