[PATCH] D47097: [DebugInfo] Preserve scope in auto generated StoreInst

[Adrian Prantl via Phabricator via cfe-commits]

aprantl added a comment.

In https://reviews.llvm.org/D47097#1111223, @gramanas wrote:

> In https://reviews.llvm.org/D47097#1111149, @probinson wrote:
>
> > Can we step back a second and better explain what the problem is? With current Clang the debug info for this function looks okay to me.
> >  The store that is "missing" a debug location is homing the formal parameter to its local stack location; this is part of prolog setup, not "real" code.
>
>
> Isn't this the reason the artificial debug loc exists? The store in the prolog might not be real code but it should still have the scope that the rest of the function has.


Instructions that are part of the function prologue are supposed to have no debug location, not an artificial one. The funciton prologue ends at the first instruction with a nonempty location.


Repository:
  rC Clang

https://reviews.llvm.org/D47097