[PATCH] D95667: [NFC][RISCV] Remove redundant pseudo instructions for vector load/store.

Jessica Clarke via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 29 04:35:50 PST 2021


jrtc27 added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVInstrInfoVPseudos.td:3165
 // Pseudos Unit-Stride Loads and Stores
-foreach eew = EEWList in {
-  defm PseudoVLE # eew : VPseudoUSLoad;
-  defm PseudoVSE # eew : VPseudoUSStore;
-}
+defm PseudoVL : VPseudoUSLoad</*fault-only-first*/false>;
+defm PseudoVS : VPseudoUSStore;
----------------
khchen wrote:
> `/*fault-only-first=*/` is correct style.
No, `/*isFF=*/`, it's always the name of the argument (and `fault-only-first` isn't even a valid identifier).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D95667



More information about the llvm-commits mailing list