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-interpreter.h | |
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-interpreter.h')
-rw-r--r-- | src/wasm-interpreter.h | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/src/wasm-interpreter.h b/src/wasm-interpreter.h index 5ca590208..d84591a00 100644 --- a/src/wasm-interpreter.h +++ b/src/wasm-interpreter.h @@ -2165,22 +2165,22 @@ private: NOTE_EVAL1(loaded); auto computed = value.getSingleValue(); switch (curr->op) { - case Add: + case RMWAdd: computed = loaded.add(computed); break; - case Sub: + case RMWSub: computed = loaded.sub(computed); break; - case And: + case RMWAnd: computed = loaded.and_(computed); break; - case Or: + case RMWOr: computed = loaded.or_(computed); break; - case Xor: + case RMWXor: computed = loaded.xor_(computed); break; - case Xchg: + case RMWXchg: break; } instance.doAtomicStore(addr, curr->bytes, computed); |