[PATCH] D58338: [WebAssembly] Refactor atomic operation definitions (NFC)

[Heejin Ahn via Phabricator via llvm-commits]

aheejin added inline comments.


================
Comment at: lib/Target/WebAssembly/WebAssemblyInstrAtomics.td:22
+
+multiclass ATOMIC_NRI<dag oops, dag iops, list<dag> pattern, string asmstr = "",
+                      bits<32> atomic_op = -1> {
----------------
tlively wrote:
> It looks like this multiclass isn't used anywhere. Can it be omitted for simplicity?
It's now used in D50277, the ever-pending CL. Do you think moving it there is better?


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D58338/new/

https://reviews.llvm.org/D58338