Enable WebAssembly.Global by default

RESOLVED FIXED in Firefox 62

Status

()

enhancement
RESOLVED FIXED
Last year
11 months ago

People

(Reporter: jseward, Assigned: jseward)

Tracking

({dev-doc-complete})

unspecified
mozilla62
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox62 fixed)

Details

Attachments

(1 attachment)

WebAssembly.Global is currently nightly- and early-beta-only, gated by
ENABLE_WASM_GLOBAL and EARLY_BETA_OR_EARLIER respectively.  This bug is to
enable it by default.
Posted patch First attemptSplinter Review
I think I got everything (famous last words, etc).
Attachment #8980971 - Flags: feedback?(lhansen)
Comment on attachment 8980971 [details] [diff] [review]
First attempt

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

Good to go, as far as I'm concerned.

::: js/src/jit-test/tests/wasm/spec/harness/index.js
@@ +224,5 @@
>      if (instance.isError())
>          return instance;
>  
>      // Experimental API change.  We try to export WebAssembly.Global instances,
>      // not primitive values.  In that case the Number() cast is necessary here

At least "Experimental API change" could be removed here but the comment block should probably be reworked to just explain what's going on.
Attachment #8980971 - Flags: feedback?(lhansen) → review+
Pushed by jseward@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/91a5de2fb8ca
Enable WebAssembly.Global by default.  r=lth.
https://hg.mozilla.org/mozilla-central/rev/91a5de2fb8ca
Status: NEW → RESOLVED
Closed: Last year
Resolution: --- → FIXED
Target Milestone: --- → mozilla62
Keywords: dev-doc-needed
Assignee: nobody → jseward
You need to log in before you can comment on or make changes to this bug.