[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