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

Adrian Prantl via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu May 24 10:12:58 PDT 2018

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.

  rC Clang


More information about the cfe-commits mailing list