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

[Orlando Cazalet-Hyams via Phabricator via llvm-commits]

Orlando marked an inline comment as done.
Orlando added a comment.

In D82129#2165231 <https://reviews.llvm.org/D82129#2165231>, @aprantl wrote:

> LGTM once Paul's comment is addressed.


Thanks! If you're referring to Paul's inline comment in test `llvm/test/DebugInfo/COFF/register-variables.ll` we resolved that in the non-inline comments and I should've marked it as done, oops. I can't find another unaddressed comment so I'll land this soon.


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

https://reviews.llvm.org/D82129