[PATCH] D151470: [RISCV][NFC] Make Reduction scheduler resources SEW aware
Michael Maitland via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu May 25 10:20:26 PDT 2023
michaelmaitland added inline comments.
================
Comment at: llvm/lib/Target/RISCV/RISCVInstrInfoVPseudos.td:3191
let isCommutable = Commutable in
- def "_" # mx # "_E" # e : VPseudoTernaryNoMaskWithPolicy<RetClass, Op1Class, Op2Class, Constraint>;
- def "_" # mx # "_E" # e # "_MASK" : VPseudoBinaryTailPolicy<RetClass, Op1Class, Op2Class, Constraint>;
- }
+ def "_" # mx # "_E" # sew : VPseudoTernaryNoMaskWithPolicy<RetClass, Op1Class, Op2Class, Constraint>;
+ def "_" # mx # "_E" # sew # "_MASK" : VPseudoBinaryTailPolicy<RetClass, Op1Class, Op2Class, Constraint>;
----------------
As a side note, are both of these records supposed to set isCommutable? It is unclear because formatting was committed incorrectly. In either case I can fix braces and/or formatting in another patch.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D151470/new/
https://reviews.llvm.org/D151470
More information about the llvm-commits
mailing list