[PATCH] D141601: [AArch64][SME2] MOVA tile-to-vector and vector-to-tile should not accept VG suffix

Caroline via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 18 07:17:34 PST 2023


CarolineConcatto added a comment.

Hey Sander,
I am not sure if this class is always used by movaz. In my search it looks it is used by them too.
I also looked at the sme2_mova_tile_or_array_to_vec_aliases, but it looks that one is fine and we don't need to change.



================
Comment at: llvm/lib/Target/AArch64/SMEInstrFormats.td:2769
        (ins tile_ty:$_ZAd, MatrixIndexGPR32Op12_15:$Rs, index_ty:$imm, vector_ty:$Zn),
-       mnemonic, "\t$ZAd[$Rs, $imm, vgx2], $Zn",
+       mnemonic, "\t$ZAd[$Rs, $imm], $Zn",
        "", []>, Sched<[]> {
----------------
Can you check if this class is also used my movaz?
Because if so, then should movaz also not use vgx?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D141601



More information about the llvm-commits mailing list