1507106 - Add atomic.notify as an alias for atomic.wake

Status: RESOLVED FIXED

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

Description

[Lars T Hansen [:lth]]

An extended bikeshedding session at the Wasm CG video call on Nov 13 decided, in the end, that we will rename atomic.wake as atomic.notify to track what JS is doing and because "wake" and "wait" sound too similar for comfort and tend to confuse listeners.  (Also lots of languages use "notify"; "wake" is a futexism.)

For the time being, let's just add an alias and not worry overmuch about updating everything else.

Comment 1

[Lars T Hansen [:lth]]

Attachment: bug1507106-atomic-notify-for-wasm.patch

Comment 2

[Julian Seward [:jseward]]

Comment on attachment 9025002 [details] [diff] [review]
bug1507106-atomic-notify-for-wasm.patch

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

Sigh (re bikeshedding etc), although I do think notify is a better name.

Yes.  Looks fine.

Comment 3

[Lars T Hansen [:lth]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/26862c8c9db982119676a63482d9ae51e59e1534
Bug 1507106 - add atomic.notify as an alias for atomic.wake. r=jseward

Comment 4

[Razvan Maries]

https://hg.mozilla.org/mozilla-central/rev/26862c8c9db9

Comment 5

[Chris Mills [:cmills]]

Not sure if this is an internals thing, or if it is exposed to web devs. Will investigate soon.

Comment 6

[Lars T Hansen [:lth]]

Well...  those few people who manipulate textual webassembly (essentially compiler writers) will be exposed to this, but it's not a JS API, it's the name of the instruction when you write wasm text.  And the browser has no means of translating wasm text to wasm binary.  So technically it's not a web API.

Comment 7

[Chris Mills [:cmills]]

OK, that makes sense; I still ought to mention it in (and generally update) the guide to WebAssembly text format on MDN, so I'll keep the dev-doc-needed keyword, but I won't include a note in the Fx65 rel notes.