diff options
author | Heejin Ahn <aheejin@gmail.com> | 2020-11-13 15:54:08 -0800 |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-11-13 15:54:08 -0800 |
commit | 75e61204b67e921464af14fd13ff768d88755e8c (patch) | |
tree | 969ddb2e4e2870fef662375914f16fc50713cd4c /src/passes/Print.cpp | |
parent | cc2b3e4175a6edc53487eec06da13b39eb66716b (diff) | |
download | binaryen-75e61204b67e921464af14fd13ff768d88755e8c.tar.gz binaryen-75e61204b67e921464af14fd13ff768d88755e8c.tar.bz2 binaryen-75e61204b67e921464af14fd13ff768d88755e8c.zip |
Rename atomic.notify and *.atomic.wait (#3353)
- atomic.notify -> memory.atomic.notify
- i32.atomic.wait -> memory.atomic.wait32
- i64.atomic.wait -> memory.atomic.wait64
See WebAssembly/threads#149.
This renames instruction name printing but not the internal data
structure names, such as `AtomicNotify`, which are not always the same
as printed instruction names anyway. This also does not modify C API.
But this fixes interface functions in binaryen.js because it seems
binaryen.js's interface functions all follow the corresponding
instruction names.
Diffstat (limited to 'src/passes/Print.cpp')
-rw-r--r-- | src/passes/Print.cpp | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/src/passes/Print.cpp b/src/passes/Print.cpp index e653e85bd..e512d398f 100644 --- a/src/passes/Print.cpp +++ b/src/passes/Print.cpp @@ -317,13 +317,15 @@ struct PrintExpressionContents } void visitAtomicWait(AtomicWait* curr) { prepareColor(o); - o << forceConcrete(curr->expectedType) << ".atomic.wait"; + Type type = forceConcrete(curr->expectedType); + assert(type == Type::i32 || type == Type::i64); + o << "memory.atomic.wait" << (type == Type::i32 ? "32" : "64"); if (curr->offset) { o << " offset=" << curr->offset; } } void visitAtomicNotify(AtomicNotify* curr) { - printMedium(o, "atomic.notify"); + printMedium(o, "memory.atomic.notify"); if (curr->offset) { o << " offset=" << curr->offset; } |