[llvm] [C API] Add getters for Target Extension Types to C API (PR #71291)
via llvm-commits
llvm-commits at lists.llvm.org
Sat Nov 4 12:42:55 PDT 2023
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-llvm-ir
Author: Benji Smith (Benjins)
<details>
<summary>Changes</summary>
These types were added in LLVM-16, and the C API supports constructing them, but not getting information back from them
This change adds getters for them to the C API, updates the echo test to be able to clone Target Extension Types, and adds some usage of them to the `echo.ll` test to confirm that they work
---
Full diff: https://github.com/llvm/llvm-project/pull/71291.diff
4 Files Affected:
- (modified) llvm/include/llvm-c/Core.h (+33)
- (modified) llvm/lib/IR/Core.cpp (+32)
- (modified) llvm/test/Bindings/llvm-c/echo.ll (+17)
- (modified) llvm/tools/llvm-c-test/echo.cpp (+36-2)
``````````diff
diff --git a/llvm/include/llvm-c/Core.h b/llvm/include/llvm-c/Core.h
index 4fc88b2b64eacea..2d95f198f64958f 100644
--- a/llvm/include/llvm-c/Core.h
+++ b/llvm/include/llvm-c/Core.h
@@ -1654,6 +1654,39 @@ LLVMTypeRef LLVMTargetExtTypeInContext(LLVMContextRef C, const char *Name,
unsigned *IntParams,
unsigned IntParamCount);
+/**
+ * Obtain the name for this target extension type
+ */
+const char *LLVMGetTargetExtTypeName(LLVMTypeRef TargetExtTy);
+
+/**
+ * Obtain the number of type parameters for this target extension type
+ */
+unsigned LLVMCountTargetExtTypeTypeParams(LLVMTypeRef TargetExtTy);
+
+/**
+ * Obtain the values of a target extension type's type parameters, output to
+ * the passed-in pointer. The pointer should have enough space for all type
+ * params for the given target extension type
+ *
+ * @see LLVMCountTargetExtTypeTypeParams
+ */
+void LLVMGetTargetExtTypeTypeParams(LLVMTypeRef TargetExtTy, LLVMTypeRef *Dest);
+
+/**
+ * Obtain the number of int parameters for this target extension type
+ */
+unsigned LLVMCountTargetExtTypeIntParams(LLVMTypeRef TargetExtTy);
+
+/**
+ * Obtain the int values of a target extension type's int parameters, output to
+ * the passed-in pointer. The pointer should have enough space for all int
+ * params for the given target extension type
+ *
+ * @see LLVMCountTargetExtTypeIntParams
+ */
+void LLVMGetTargetExtTypeIntParams(LLVMTypeRef TargetExtTy, unsigned *Dest);
+
/**
* @}
*/
diff --git a/llvm/lib/IR/Core.cpp b/llvm/lib/IR/Core.cpp
index 076d1089582fe7e..eb1cda55288e334 100644
--- a/llvm/lib/IR/Core.cpp
+++ b/llvm/lib/IR/Core.cpp
@@ -924,6 +924,38 @@ LLVMTypeRef LLVMTargetExtTypeInContext(LLVMContextRef C, const char *Name,
TargetExtType::get(*unwrap(C), Name, TypeParamArray, IntParamArray));
}
+const char *LLVMGetTargetExtTypeName(LLVMTypeRef TargetExtTy) {
+ TargetExtType *Type = unwrap<TargetExtType>(TargetExtTy);
+ return Type->getName().data();
+}
+
+unsigned LLVMCountTargetExtTypeTypeParams(LLVMTypeRef TargetExtTy) {
+ TargetExtType *Type = unwrap<TargetExtType>(TargetExtTy);
+ return Type->getNumTypeParameters();
+}
+
+void LLVMGetTargetExtTypeTypeParams(LLVMTypeRef TargetExtTy,
+ LLVMTypeRef *Dest) {
+ TargetExtType *Type = unwrap<TargetExtType>(TargetExtTy);
+
+ for (unsigned int i = 0; i < Type->getNumTypeParameters(); i++) {
+ Dest[i] = wrap(Type->getTypeParameter(i));
+ }
+}
+
+unsigned LLVMCountTargetExtTypeIntParams(LLVMTypeRef TargetExtTy) {
+ TargetExtType *Type = unwrap<TargetExtType>(TargetExtTy);
+ return Type->getNumIntParameters();
+}
+
+void LLVMGetTargetExtTypeIntParams(LLVMTypeRef TargetExtTy, unsigned *Dest) {
+ TargetExtType *Type = unwrap<TargetExtType>(TargetExtTy);
+
+ for (unsigned int i = 0; i < Type->getNumIntParameters(); i++) {
+ Dest[i] = Type->getIntParameter(i);
+ }
+}
+
/*===-- Operations on values ----------------------------------------------===*/
/*--.. Operations on all values ............................................--*/
diff --git a/llvm/test/Bindings/llvm-c/echo.ll b/llvm/test/Bindings/llvm-c/echo.ll
index 5daa238bfb8e533..e62670424112d51 100644
--- a/llvm/test/Bindings/llvm-c/echo.ll
+++ b/llvm/test/Bindings/llvm-c/echo.ll
@@ -66,6 +66,23 @@ define void @types() {
ret void
}
+; Target extension types:
+define target("target.ext.1") @target_ext_01(target("target.ext.1") %0) {
+ ret target("target.ext.1") %0
+}
+
+define target("target.ext.2", i8, i1) @target_ext_02(target("target.ext.2", i8, i1) %0) {
+ ret target("target.ext.2", i8, i1) %0
+}
+
+define target("target.ext.3", 7) @target_ext_03(target("target.ext.3", 7) %0) {
+ ret target("target.ext.3", 7) %0
+}
+
+define target("target.ext.4", i1, i32, 7) @target_ext_04(target("target.ext.4", i1, i32, 7) %0) {
+ ret target("target.ext.4", i1, i32, 7) %0
+}
+
define i32 @iops(i32 %a, i32 %b) {
%1 = add i32 %a, %b
%2 = mul i32 %a, %1
diff --git a/llvm/tools/llvm-c-test/echo.cpp b/llvm/tools/llvm-c-test/echo.cpp
index 06966ce528eae4d..8ff7bc1ed0bf485 100644
--- a/llvm/tools/llvm-c-test/echo.cpp
+++ b/llvm/tools/llvm-c-test/echo.cpp
@@ -157,8 +157,42 @@ struct TypeCloner {
return LLVMX86MMXTypeInContext(Ctx);
case LLVMTokenTypeKind:
return LLVMTokenTypeInContext(Ctx);
- case LLVMTargetExtTypeKind:
- assert(false && "Implement me");
+ case LLVMTargetExtTypeKind: {
+ const char *Name = LLVMGetTargetExtTypeName(Src);
+ unsigned TypeParamCount = LLVMCountTargetExtTypeTypeParams(Src);
+ unsigned IntParamCount = LLVMCountTargetExtTypeIntParams(Src);
+
+ LLVMTypeRef *TypeParams = nullptr;
+ unsigned *IntParams = nullptr;
+
+ // If we have type params, get them from Src and clone them individually
+ if (TypeParamCount > 0) {
+ TypeParams = static_cast<LLVMTypeRef *>(
+ safe_malloc(TypeParamCount * sizeof(LLVMTypeRef)));
+ LLVMGetTargetExtTypeTypeParams(Src, TypeParams);
+
+ for (unsigned i = 0; i < TypeParamCount; i++)
+ TypeParams[i] = Clone(TypeParams[i]);
+ }
+
+ // If we have integer params, get them from Src
+ if (IntParamCount > 0) {
+ IntParams = static_cast<unsigned *>(
+ safe_malloc(IntParamCount * sizeof(unsigned)));
+ LLVMGetTargetExtTypeIntParams(Src, IntParams);
+ }
+
+ LLVMTypeRef TargtExtTy = LLVMTargetExtTypeInContext(
+ Ctx, Name, TypeParams, TypeParamCount, IntParams, IntParamCount);
+
+ if (TypeParams)
+ free(TypeParams);
+
+ if (IntParams)
+ free(IntParams);
+
+ return TargtExtTy;
+ }
}
fprintf(stderr, "%d is not a supported typekind\n", Kind);
``````````
</details>
https://github.com/llvm/llvm-project/pull/71291
More information about the llvm-commits
mailing list