[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