[llvm] [IPSCCP] Variable not visible at Og. (PR #66745)

J. Ryan Stinnett via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 9 06:06:38 PDT 2023


jryans wrote:

> I still think that the logic for "convert Constant into DIExpression" should not be part of IPSCCP, but be a helper in some generic location (not sure where, debuginfo folks would know). This seems generically useful (to the point that I'm surprised it does not exist yet).

I agree, it does seem good to make this available as a shared helper. Perhaps it could go to https://github.com/llvm/llvm-project/blob/main/llvm/lib/Transforms/Utils/Local.cpp? There are various shared debug info helpers stashed there already. (At some stage, a separate file just for debug info utils might be a good idea, but that can be separate task.)

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


More information about the llvm-commits mailing list