[llvm] [RemoveDIs] Enable conversion from dbg.declare to DPValue (PR #74090)
Jeremy Morse via llvm-commits
llvm-commits at lists.llvm.org
Mon Dec 11 09:47:12 PST 2023
https://github.com/jmorse approved this pull request.
Overall idea LGTM. In terms of landing things, we've made a mess by having this partial overlapping coverage sadly, so there'll need to be some kind of gap for some period.
An additional constraint however is that there /might/ now be people testing this code, seeing how we've announced testing, and if we land the dbg-declare-conversion code up front, there might be some spurious differences / crashes / whatever in the meantime. I'd prefer option 2 to avoid punishing anyone who's kind enough to test for us.
(Or add another command line flag that lasts two days and turns on dbg-declare conversion? All bad options sorry).
https://github.com/llvm/llvm-project/pull/74090
More information about the llvm-commits
mailing list