[llvm] [DebugInfo] getMergedLocation: match scopes based on their location (PR #132286)

Stephen Tozer via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 4 04:56:23 PDT 2025


https://github.com/SLTozer commented:

I think I'm more confident with the direction of this patch now - thanks for your patience  with the updates. Code changes largely LGTM at this point too. Just double checking: @dwblaikie are you happy with the approach described above? I think this gives us strictly more information, and doesn't give us anything that's "incorrect"; hopefully it doesn't result in generating too many new scopes, but the logic seems like it's rarely applicable to most languages at least.

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


More information about the llvm-commits mailing list