[PATCH] D139950: [WebAssembly] Use ComplexPattern on store address operands

Luke Lau via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 14 02:51:54 PST 2022


luke added inline comments.


================
Comment at: llvm/lib/Target/WebAssembly/WebAssemblyInstrMemory.td:24-26
+// These patterns match the static (offset) and dynamic (address stack operand)
+// operands for loads and stores, based on a combination of target global
+// addresses and constants.
----------------
aheejin wrote:
> Nit: 80 col wrapping
I think this is wrapped to 80 columns, unfortunately just a poor choice of long, awkward to wrap words :(


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D139950



More information about the llvm-commits mailing list