-
Notifications
You must be signed in to change notification settings - Fork 50
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Rename *.atomic.wake to *.atomic.notify? #104
Comments
As a counterpoint, OS APIs seem to tend towards "wake", either in name or documentation (wake_on_bit), (interrupting threads) and we've already diverged in convention from JS slightly on wait/wake to be closer to the OS (#36). Caught between two worlds! |
We named those instructions in |
(I don't have an opinion on which names, but with @aardappel's work, the LLVM wasm backend has been moving in the direction of harmonizing its text assembly format with the official wat format, so we really should use the same names, whatever they are, since that's the easy part :-)). |
I have a slight preference for keeping |
I guess we can close it now, after we decided to use |
JavaScript recently renamed
Atomics.wake
toAtomics.notify
, see tc39/ecma262#1220. Should we do the same for the WebAssembly atomics instructions?The text was updated successfully, but these errors were encountered: