[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