[llvm] [RemoveDIs][DebugInfo] Create overloads of debug intrinsic utilities for DPValues (PR #78313)

Jeremy Morse via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 16 10:27:18 PST 2024


https://github.com/jmorse approved this pull request.

This LGTM; I think I'd normally frown slightly on pushing all of this abstraction out into overloaded functions, however I reckon it's a proportionate response to reducing the amount of modifications needed to debug-info-updating code. We'll only be inevitably injecting bugs otherwise.

Plus, in some currently unthought future, where dbg.value intrinsics are not just gone but also the class hierachy, we can just fold these abstractions back into the code using them.

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


More information about the llvm-commits mailing list