[PATCH] D96567: [RISCV] Add support for fixed vector floating point setcc.
Craig Topper via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Feb 12 09:54:05 PST 2021
craig.topper added inline comments.
================
Comment at: llvm/lib/Target/RISCV/RISCVInstrInfoVVLPatterns.td:84
+ SDTypeProfile<1, 5, [SDTCisVec<0>,
+ SDTCVecEltisVT<0, i1>,
+ SDTCisVec<1>,
----------------
frasercrmck wrote:
> I was wondering if this warrants a `SDTCisVecWithSameNumEltsAndEltVT<opidx, otheropidx, eltvt>` to combine these three commonly-used constraints (`SDTCisVec<1>, SDTCisSameNumEltsAs<0, 1>, SDTCVecEltisVT<0, X>`) but the name alone gives me second thoughts.
I think the tablegen backend maps the name of the class to dispatch to a function that does the type constraining.
I suspect SDTCisVec is redundant if SDTCVecEltisVT is specified.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D96567/new/
https://reviews.llvm.org/D96567
More information about the llvm-commits
mailing list