[PATCH] D95979: [RISCV] Split vrgather intrinsics into separate vrgather.vv and vrgather.vx intrinsics.
Fraser Cormack via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Feb 4 01:22:08 PST 2021
frasercrmck added a comment.
Seems reasonable to me, but given I dip in and out of the intrinsics layer I might wait for another review.
================
Comment at: llvm/include/llvm/IR/IntrinsicsRISCV.td:249
+ // Input: (vector_in, vector_in, mask, vl)
+ class RISCVBinaryAAM
: Intrinsic<[llvm_anyvector_ty],
----------------
nit, but `M` seems out of place given that the others are still `Mask`?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D95979/new/
https://reviews.llvm.org/D95979
More information about the llvm-commits
mailing list