[clang] [llvm] [DebugInfo] Update DIBuilder insertion to take InsertPosition (PR #126059)

Jeremy Morse via cfe-commits cfe-commits at lists.llvm.org
Fri Feb 7 08:07:36 PST 2025


https://github.com/jmorse commented:

This looks great, thanks for putting this all together, one nit inline,

Is this intended to be cherry-picked into the release branch or not? Seeing how the original motivation was to ease downstream compatibilities, I'd assumed there'd be the need for something to be cherry-picked, and this might be too invasive at this stage to be moved across. I don't have a full understanding of the use case though.

@OCHyams I believe you did the C API changes, are there any additional concerns in this area?

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


More information about the cfe-commits mailing list