[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