[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