[PATCH] D82129: [DebugInfo] Drop location ranges for variables which exist entirely outside the variable's scope

Djordje Todorovic via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 25 02:08:43 PDT 2020


djtodoro added a comment.

In D82129#2111602 <https://reviews.llvm.org/D82129#2111602>, @probinson wrote:

> In D82129#2110934 <https://reviews.llvm.org/D82129#2110934>, @Orlando wrote:
>
> > In D82129#2107017 <https://reviews.llvm.org/D82129#2107017>, @probinson wrote:
> >
> > > This patch depends on the ranges for all scopes to be (reasonably) correct,
> >
> >
> > I'd say instead that 'variable locations depend on the ranges for all scopes to be (reasonably) correct'. And that this patch just acknowledges that relationship and clears away what we cannot use/see in a debugger.
>
>
> Either way, the question remains: when we find cases where we need to "clear away" something, is that a bug, or is this merely a cleanup pass?  In the case of the test I commented on, it's a bug, and I'd rather not be hiding bugs.


+1.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D82129/new/

https://reviews.llvm.org/D82129





More information about the llvm-commits mailing list