[PATCH] D80769: [WebAssembly] Adding 64-bit versions of all load & store ops.

Thomas Lively via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 12 18:48:37 PDT 2020


tlively added inline comments.


================
Comment at: llvm/lib/Target/WebAssembly/WebAssemblyInstrMemory.td:371
                          "memory.size\t$dst, $flags", "memory.size\t$flags",
-                         0x3f>,
-                       Requires<[HasAddr32]>;
+                         0x3f>;
 
----------------
dschuff wrote:
> Question for the tblgen experts maybe,  if a def has both an instruction definition and a pattern as this one does, does the `Requires` field actually apply to the pattern? Or does  `Requires` only work for freestanding patterns?
I would have thought that `Requires` should apply to both kinds of patterns. In particular, inheriting it just sets the `Predicates` field in both `Instruction` and `Pattern`. But I read @aheejin's comment above to mean that this doesn't work after all, so perhaps I am wrong.


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

https://reviews.llvm.org/D80769





More information about the llvm-commits mailing list