eopXD 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,
----------------
craig.topper wrote:
> 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.
This won't affect the front-end. C intrinsics are directly lowered into LLVM 
intrinsics here.


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