1438113 - Make it possible to have wasm baseline without wasm ion

Status: RESOLVED FIXED

(Core :: JavaScript Engine: JIT, enhancement, P3)

Description

[Lars T Hansen [:lth]]

For ARM64 we'll initially only have baseline, and we'll be in the situation where we can support wasm but not asm.js, and we can't use ion as a fallback if the baseline compiler has been disabled by pref or or command line option.  Previously we could have ion without baseline; we just need to make the situation symmetric.

Comment 1

[Lars T Hansen [:lth]]

Attachment: bug1438113-optional-ion.patch

Make compiler selection more subtle.  See commit msg for more.

Comment 2

[Lars T Hansen [:lth]]

Comment on attachment 8950860 [details] [diff] [review]
bug1438113-optional-ion.patch

Make compiler selection more subtle.

Comment 3

[Luke Wagner [:luke]]

Comment on attachment 8950860 [details] [diff] [review]
bug1438113-optional-ion.patch

Review of attachment 8950860 [details] [diff] [review]:
-----------------------------------------------------------------

Makes sense

Comment 4

[Lars T Hansen [:lth]]

Attachment: bug1438113-optional-ion-v2.patch

Rebased, etc, ready to land.  Carrying r+.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/6e46ff4ff446
Make it possible to have wasm baseline without ion. r=luke

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/ef89c3c179cd
add missing include files, r=bustage ON CLOSED TREE

Comment 7

[Tiberius Oros[:tiberius_oros]]

https://hg.mozilla.org/mozilla-central/rev/6e46ff4ff446
https://hg.mozilla.org/mozilla-central/rev/ef89c3c179cd