[llvm] [SPIR-V] Fix types of internal intrinsic functions and add a test case for __builtin_alloca() (PR #92265)

Michal Paszkowski via llvm-commits llvm-commits at lists.llvm.org
Thu May 16 22:28:23 PDT 2024


================
@@ -27,7 +27,7 @@ let TargetPrefix = "spv" in {
   def int_spv_insertv : Intrinsic<[llvm_i32_ty], [llvm_i32_ty, llvm_any_ty, llvm_vararg_ty]>;
   def int_spv_extractelt : Intrinsic<[llvm_any_ty], [llvm_any_ty, llvm_anyint_ty]>;
   def int_spv_insertelt : Intrinsic<[llvm_any_ty], [llvm_any_ty, llvm_any_ty, llvm_anyint_ty]>;
-  def int_spv_const_composite : Intrinsic<[llvm_i32_ty], [llvm_vararg_ty]>;
+  def int_spv_const_composite : Intrinsic<[llvm_any_ty], [llvm_vararg_ty]>;
----------------
michalpaszkowski wrote:

Please also change the description in [SPIRVUsage](https://llvm.org/docs/SPIRVUsage.html#target-intrinsics) file. It might make sense to add a comment somewhere in IntrinsicsSPIRV.td which will signal that any modifications/new intrinsics need to be also documented in SPIRVUsage.

https://github.com/llvm/llvm-project/pull/92265


More information about the llvm-commits mailing list