[llvm] [RemoveDIs] Update DIBuilder C API with DbgRecord functions. (PR #95535)

Orlando Cazalet-Hyams via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 14 09:01:18 PDT 2024


OCHyams wrote:

> Is this removing any C APIs that existed in the previous release? That's a no-no.

Yes and no. It's removing some functions that I added during this release (unambiguously OK) but it's also removing some functions that insert debug intrinsics (as opposed to debug records) in accordance with the discussion linked by Carlos in the summary (https://github.com/llvm/llvm-project/pull/92417#issuecomment-2120078326 is the important one).

It was discussed whether the existing functions should be deprecated first, but @nikic voted in favour of deleting them.

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


More information about the llvm-commits mailing list