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

David Blaikie via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 4 08:39:46 PDT 2025


dwblaikie wrote:

> 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.

yeah, thanks for taking a look - I'm OK whenever you're OK with this, given the current state/direction.

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


More information about the llvm-commits mailing list