[PATCH] D119535: [VP] vp.fptosi cast intrinsic and docs

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 11 10:19:52 PST 2022


craig.topper added inline comments.


================
Comment at: llvm/include/llvm/IR/Intrinsics.td:1523
+def int_vp_fptosi : DefaultAttrsIntrinsic<[ llvm_anyvector_ty ],
+                            [ llvm_anyvector_ty,
+                              LLVMScalarOrSameVectorWidth<0, llvm_i1_ty>,
----------------
Should we have a Verifier check that the number of elements match? The equivalent constrained intrinsics check that.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D119535



More information about the llvm-commits mailing list