[RemoveDIs] Add release note for C API changes #92396
Conversation
| * ``LLVMDIBuilderInsertDeclareRecordBefore`` | ||
| * ``LLVMDIBuilderInsertDeclareRecordAtEnd`` | ||
| * ``LLVMDIBuilderInsertDbgValueRecordBefore`` | ||
| * ``LLVMDIBuilderInsertDbgValueRecordAtEnd`` |
There was a problem hiding this comment.
Is it possible to remove these prior to the LLVM 19 release? We shouldn't add temporary functions to the C API if we can avoid it.
There was a problem hiding this comment.
I think we could remove the "InsertDbg...Record" four above without a problem, since these are just aliases for LLVMDIBuilderInsertDeclareBefore and friends (the modified ones below).
It's more difficult to remove the intrinsic based insertion functions, e.g. LLVMDIBuilderInsertDeclareIntrinsicBefore, because these return a different type to the debug record insertion ones.
All of these functions above have been added as a transition aid for projects that aren't ready to move to debug records (the new debug info format). Most projects shouldn't need to use them. For projects ready to use debug intrinsics the only update required would be to address compiler errors introduced by the new return type of the modified functions below.
|
Closing as the docs updates were handled by @CarlosAlbertoEnciso in #95535 |
The changes have been documented in the migration guide llvm/docs/RemoveDIsDebugInfo.md - add the info to the release notes.