[PATCH] D86633: [MC][SVE] Fix data operand for instruction alias of `st1d`.

Paul Walker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 26 09:14:52 PDT 2020


paulwalker-arm added inline comments.


================
Comment at: llvm/test/MC/AArch64/SVE/st1d.s:82
+
+st1d    { z0.d }, p7, [z0.d, #0]
+// CHECK-INST: st1d    { z0.d }, p7, [z0.d]
----------------
Does this mean we're also missing test coverage for st1b, st1h & st1w variants?

Is it worth also testing `st1d    { z0.d }, p7, [z0.d]` directly.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86633



More information about the llvm-commits mailing list