[PATCH] Do not attach a debug location to code inserted by ARC / API for disabling DebugLocations

[John McCall]

On Apr 8, 2013, at 11:06 AM, Eric Christopher <echristo at gmail.com> wrote:
> On Mon, Apr 8, 2013 at 10:40 AM, John McCall <rjmccall at apple.com> wrote:
>> On Apr 5, 2013, at 5:22 PM, Adrian Prantl <aprantl at apple.com> wrote:
>>> Patch for review, mostly for the IRBuilder::DisableDebugLocations() part.
>>> 
>>> Do not attach a debug location to code inserted by ARC --
>>> it would create a spurious line table entry at the closing } of the scope.
>> 
>> Is this a problem?  Just that we don't want "next" to stop here?
> 
> Pretty much. It's analogous to the same when we're looking at
> cleanups, etc. I think this is going into the realm of "opinions on
> behavior" here.

Is 'next' driven completely by line table information, or is there some
way in DWARF to more explicitly say "this is a point that 'next' should
stop at"?

John.