[llvm] [RemoveDIs] Update DIBuilder C API with DbgRecord functions [2/2] (PR #85657)

Stephen Tozer via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 22 08:13:55 PDT 2024


SLTozer wrote:

SGTM, I'd like to add more detail but I don't think we can fit it all in an assert; optionally, a comment line above to the effect of "Verify we have the right format, see `LLVMIsNewDbgInfoFormat`" or such. Worth demonstrating extra caution here, since we're potentially impacting people who're disconnected from the RemoveDI work.

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


More information about the llvm-commits mailing list