1654813 - Cranelift: use the new-backend by default on x86_64

Status: RESOLVED FIXED

(Core :: JavaScript: WebAssembly, enhancement, P3)

Comment 1

[Benjamin Bouvier [:bbouvier] (inactive)]

Attachment: Bug 1654813 - Bump Cranelift to dd098656111396afa58e90084a705744e836bf10; r=cfallin

This is waiting on https://github.com/bytecodealliance/wasmtime/pull/2068 to be
landed first.

Comment 2

[Benjamin Bouvier [:bbouvier] (inactive)]

Attachment: Bug 1654813: Use the new backend for x86_64 wasm compilation in Cranelift; r=cfallin

This removes the expensive x86 "meta" build step, so this should speed up
Firefox compile times, and it also removes support for ye olde Cranelift
backend, which is a nice win.

Comment 3

[Benjamin Bouvier [:bbouvier] (inactive)]

Eventually identified the Godot failure, which was a nasty bug (thanks Chris for fixing it!).

Here's a green try build (including perfherder runs of Godot/embenchen):
https://treeherder.mozilla.org/#/jobs?repo=try&revision=b21db0e2efc11ade79498b2c6439e0a081a46f98

So this could land once https://github.com/bytecodealliance/wasmtime/pull/2083 lands upstream, and a new bump with this commit lands here.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/e396a7a097c8
Bump Cranelift to dd098656111396afa58e90084a705744e836bf10; r=cfallin
https://hg.mozilla.org/integration/autoland/rev/ef4f81977f92
Use the new backend for x86_64 wasm compilation in Cranelift; r=cfallin

Comment 5

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

https://hg.mozilla.org/mozilla-central/rev/e396a7a097c8
https://hg.mozilla.org/mozilla-central/rev/ef4f81977f92