From 35276f16e5a2cae0dfb49c0fbf874d4d2f177acc Mon Sep 17 00:00:00 2001 From: Meghan Denny Date: Wed, 15 Feb 2023 09:47:32 +0100 Subject: [PATCH] [llvm-c] Add C API methods to match 64bit ArrayType C++ API signatures Fixes https://github.com/llvm/llvm-project/issues/56496. As mentioned in the issue, new functions LLVMArrayType2 and LLVMGetArrayLength2 are created so as to not break the old API. The old methods are then marked as deprecated and callers are updated. Differential Revision: https://reviews.llvm.org/D143700 --- llvm/bindings/ocaml/llvm/llvm_ocaml.c | 2 +- llvm/include/llvm-c/Core.h | 33 +++++++++++++++++++++++++++++++++ llvm/lib/IR/Core.cpp | 14 ++++++++++++++ llvm/tools/llvm-c-test/echo.cpp | 10 ++++------ 4 files changed, 52 insertions(+), 7 deletions(-) diff --git a/llvm/bindings/ocaml/llvm/llvm_ocaml.c b/llvm/bindings/ocaml/llvm/llvm_ocaml.c index f193fe7..0762ebe 100644 --- a/llvm/bindings/ocaml/llvm/llvm_ocaml.c +++ b/llvm/bindings/ocaml/llvm/llvm_ocaml.c @@ -587,7 +587,7 @@ LLVMTypeRef llvm_vector_type(LLVMTypeRef ElementTy, value Count) { /* lltype -> int */ value llvm_array_length(LLVMTypeRef ArrayTy) { - return Val_int(LLVMGetArrayLength(ArrayTy)); + return Val_int(LLVMGetArrayLength2(ArrayTy)); } /* lltype -> int */ diff --git a/llvm/include/llvm-c/Core.h b/llvm/include/llvm-c/Core.h index 94a25ff..df9db59 100644 --- a/llvm/include/llvm-c/Core.h +++ b/llvm/include/llvm-c/Core.h @@ -1437,20 +1437,43 @@ unsigned LLVMGetNumContainedTypes(LLVMTypeRef Tp); * The created type will exist in the context that its element type * exists in. * + * @deprecated LLVMArrayType is deprecated in favor of the API accurate + * LLVMArrayType2 * @see llvm::ArrayType::get() */ LLVMTypeRef LLVMArrayType(LLVMTypeRef ElementType, unsigned ElementCount); /** + * Create a fixed size array type that refers to a specific type. + * + * The created type will exist in the context that its element type + * exists in. + * + * @see llvm::ArrayType::get() + */ +LLVMTypeRef LLVMArrayType2(LLVMTypeRef ElementType, uint64_t ElementCount); + +/** * Obtain the length of an array type. * * This only works on types that represent arrays. * + * @deprecated LLVMGetArrayLength is deprecated in favor of the API accurate + * LLVMGetArrayLength2 * @see llvm::ArrayType::getNumElements() */ unsigned LLVMGetArrayLength(LLVMTypeRef ArrayTy); /** + * Obtain the length of an array type. + * + * This only works on types that represent arrays. + * + * @see llvm::ArrayType::getNumElements() + */ +uint64_t LLVMGetArrayLength2(LLVMTypeRef ArrayTy); + +/** * Create a pointer type that points to a defined type. * * The created type will exist in the context that its pointee type @@ -2118,12 +2141,22 @@ LLVMValueRef LLVMConstStruct(LLVMValueRef *ConstantVals, unsigned Count, /** * Create a ConstantArray from values. * + * @deprecated LLVMConstArray is deprecated in favor of the API accurate + * LLVMConstArray2 * @see llvm::ConstantArray::get() */ LLVMValueRef LLVMConstArray(LLVMTypeRef ElementTy, LLVMValueRef *ConstantVals, unsigned Length); /** + * Create a ConstantArray from values. + * + * @see llvm::ConstantArray::get() + */ +LLVMValueRef LLVMConstArray2(LLVMTypeRef ElementTy, LLVMValueRef *ConstantVals, + uint64_t Length); + +/** * Create a non-anonymous ConstantStruct from values. * * @see llvm::ConstantStruct::get() diff --git a/llvm/lib/IR/Core.cpp b/llvm/lib/IR/Core.cpp index a93fc85..a0fade9 100644 --- a/llvm/lib/IR/Core.cpp +++ b/llvm/lib/IR/Core.cpp @@ -788,6 +788,10 @@ LLVMTypeRef LLVMArrayType(LLVMTypeRef ElementType, unsigned ElementCount) { return wrap(ArrayType::get(unwrap(ElementType), ElementCount)); } +LLVMTypeRef LLVMArrayType2(LLVMTypeRef ElementType, uint64_t ElementCount) { + return wrap(ArrayType::get(unwrap(ElementType), ElementCount)); +} + LLVMTypeRef LLVMPointerType(LLVMTypeRef ElementType, unsigned AddressSpace) { return wrap(PointerType::get(unwrap(ElementType), AddressSpace)); } @@ -822,6 +826,10 @@ unsigned LLVMGetArrayLength(LLVMTypeRef ArrayTy) { return unwrap(ArrayTy)->getNumElements(); } +uint64_t LLVMGetArrayLength2(LLVMTypeRef ArrayTy) { + return unwrap(ArrayTy)->getNumElements(); +} + unsigned LLVMGetPointerAddressSpace(LLVMTypeRef PointerTy) { return unwrap(PointerTy)->getAddressSpace(); } @@ -1493,6 +1501,12 @@ LLVMValueRef LLVMConstArray(LLVMTypeRef ElementTy, return wrap(ConstantArray::get(ArrayType::get(unwrap(ElementTy), Length), V)); } +LLVMValueRef LLVMConstArray2(LLVMTypeRef ElementTy, LLVMValueRef *ConstantVals, + uint64_t Length) { + ArrayRef V(unwrap(ConstantVals, Length), Length); + return wrap(ConstantArray::get(ArrayType::get(unwrap(ElementTy), Length), V)); +} + LLVMValueRef LLVMConstStructInContext(LLVMContextRef C, LLVMValueRef *ConstantVals, unsigned Count, LLVMBool Packed) { diff --git a/llvm/tools/llvm-c-test/echo.cpp b/llvm/tools/llvm-c-test/echo.cpp index 7a54e26..ed2999f 100644 --- a/llvm/tools/llvm-c-test/echo.cpp +++ b/llvm/tools/llvm-c-test/echo.cpp @@ -133,10 +133,8 @@ struct TypeCloner { return S; } case LLVMArrayTypeKind: - return LLVMArrayType( - Clone(LLVMGetElementType(Src)), - LLVMGetArrayLength(Src) - ); + return LLVMArrayType2(Clone(LLVMGetElementType(Src)), + LLVMGetArrayLength2(Src)); case LLVMPointerTypeKind: if (LLVMPointerTypeIsOpaque(Src)) return LLVMPointerTypeInContext(Ctx, LLVMGetPointerAddressSpace(Src)); @@ -309,9 +307,9 @@ static LLVMValueRef clone_constant_impl(LLVMValueRef Cst, LLVMModuleRef M) { ? LLVMConstantArrayValueKind : LLVMConstantDataArrayValueKind); LLVMTypeRef Ty = TypeCloner(M).Clone(Cst); - unsigned EltCount = LLVMGetArrayLength(Ty); + uint64_t EltCount = LLVMGetArrayLength2(Ty); SmallVector Elts; - for (unsigned i = 0; i < EltCount; i++) + for (uint64_t i = 0; i < EltCount; i++) Elts.push_back(clone_constant(LLVMGetAggregateElement(Cst, i), M)); return LLVMConstArray(LLVMGetElementType(Ty), Elts.data(), EltCount); } -- 2.7.4