[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