[llvm] [LLVM] Add `LLVMIntrinsicCopyName` to LLVM C API (PR #112197)

via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 14 11:22:56 PDT 2024


================
@@ -242,6 +242,10 @@ Changes to the C API
 
 * Added `LLVMAtomicRMWBinOpUSubCond` and `LLVMAtomicRMWBinOpUSubSat` to `LLVMAtomicRMWBinOp` enum for AtomicRMW instructions.
 
+* Added `LLVMIntrinsicCopyName` and changed `LLVMIntrinsicCopyOverloadedName`
+  and `LLVMIntrinsicCopyOverloadedName2` to return `const char *` instead of 
+  `const char *`.
----------------
graphite-app[bot] wrote:

The release notes contain an inaccuracy regarding the return types of `LLVMIntrinsicCopyOverloadedName` and `LLVMIntrinsicCopyOverloadedName2`. They now return `char *` instead of `const char *`, not the other way around as currently stated. This change is significant as it reflects the caller's ownership of the returned memory. Please update the release notes to correctly describe this modification.

*Spotted by [Graphite Reviewer](https://app.graphite.dev/graphite-reviewer/?org=llvm&ref=ai-review-comment)*<i class='graphite__hidden'><br /><br />Is this helpful? React 👍 or 👎 to let us know.</i>

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


More information about the llvm-commits mailing list