[PATCH] D105351: [VP] Declaration and docs for vp.select intrinsic

[Roger Ferrer Ibanez via Phabricator via llvm-commits]

rogfer01 added inline comments.


================
Comment at: llvm/include/llvm/IR/Intrinsics.td:1511
+// Shuffles.
+def int_vp_select : DefaultAttrsIntrinsic<[ llvm_anyvector_ty ],
+                              [ LLVMScalarOrSameVectorWidth<0, llvm_i1_ty>,
----------------
Deep apologies because I didn't realise this earlier: shouldn't this intrinsic be at least `IntrNoMem`? It was in the reference patch and it'll probably make a difference when you add `VP_SELECT`.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D105351/new/

https://reviews.llvm.org/D105351