diff options
| author | Thomas Lively <7121787+tlively@users.noreply.github.com> | 2020-10-26 10:31:05 -0700 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2020-10-26 10:31:05 -0700 |
| commit | 019b0ef15e966ecafa6cdfa357b00de09c332633 (patch) | |
| tree | e9fd5a0d8b130168280683f71fecbc92bea4a9d7 /src/wasm/wasm-stack.cpp | |
| parent | d0032c299695c765e757de5f9482a610bf687d69 (diff) | |
| download | binaryen-019b0ef15e966ecafa6cdfa357b00de09c332633.tar.gz binaryen-019b0ef15e966ecafa6cdfa357b00de09c332633.tar.bz2 binaryen-019b0ef15e966ecafa6cdfa357b00de09c332633.zip | |
[NFC] `using namespace Abstract` to make matchers more compact (#3284)
This change makes matchers in OptimizeInstructions more compact and readable by
removing the explicit `Abstract::` namespace from individual operations. In some
cases, this makes multi-line matcher expressions fit on a single line.
This change is only possible because it also adds an explicit "RMW" prefix to
each element of the `AtomicRMWOp` enumeration. Without that, their names
conflicted with the names of Abstract ops.
Diffstat (limited to 'src/wasm/wasm-stack.cpp')
| -rw-r--r-- | src/wasm/wasm-stack.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/wasm/wasm-stack.cpp b/src/wasm/wasm-stack.cpp index 4bc479d15..13ce700a5 100644 --- a/src/wasm/wasm-stack.cpp +++ b/src/wasm/wasm-stack.cpp @@ -351,7 +351,7 @@ void BinaryInstWriter::visitAtomicRMW(AtomicRMW* curr) { o << int8_t(BinaryConsts::AtomicPrefix); #define CASE_FOR_OP(Op) \ - case Op: \ + case RMW##Op: \ switch (curr->type.getBasic()) { \ case Type::i32: \ switch (curr->bytes) { \ |