craig.topper added inline comments.

================
Comment at: llvm/include/llvm/IR/IntrinsicsRISCV.td:697
+        : DefaultAttrsIntrinsic<[llvm_anyvector_ty],
+                    [LLVMMatchType<0>, llvm_anyvector_ty, llvm_any_ty,
+                     LLVMScalarOrSameVectorWidth<0, llvm_i1_ty>, 
llvm_anyint_ty,
----------------
Something I just noticed today. I think we can use `LLVMExtendedType<0>` 
instead of llvm_any_vector_ty for the second input operand to make it 
automatically 2x the element width of the output. Does that simplify the 
frontend much if we don't need to use an extra IntrinsicType? It would be a 
separate patch of course if we did it.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D152879

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to