Enable WebAssembly.Global by default

RESOLVED FIXED in Firefox 62

Status

()

enhancement
RESOLVED FIXED
a year ago
9 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)

Assignee

Description

a year ago
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.
Assignee

Comment 1

a year ago
Posted patch First attemptSplinter Review
I think I got everything (famous last words, etc).
Assignee

Updated

a year ago
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+

Comment 4

a year ago
Pushed by jseward@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/91a5de2fb8ca
Enable WebAssembly.Global by default.  r=lth.

Comment 5

a year ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/91a5de2fb8ca
Status: NEW → RESOLVED
Last Resolved: a year ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla62
Assignee

Updated

a year ago
Keywords: dev-doc-needed
Assignee: nobody → jseward
You need to log in before you can comment on or make changes to this bug.