[llvm] r212205 - DebugInfo: Ensure that all debug location scope chains from instructions within a function, lead to the function itself.

[David Blaikie]

Reverted in r212226 - would still be interested in any test cases.
I've got a pretty good creduce setup going, so it doesn't need to be
minimized, just reproducible.

On Wed, Jul 2, 2014 at 2:36 PM, David Blaikie <dblaikie at gmail.com> wrote:
> On Wed, Jul 2, 2014 at 2:35 PM, David Blaikie <dblaikie at gmail.com> wrote:
>> On Wed, Jul 2, 2014 at 2:29 PM, Reid Kleckner <rnk at google.com> wrote:
>>> On Wed, Jul 2, 2014 at 11:32 AM, David Blaikie <dblaikie at gmail.com> wrote:
>>>>
>>>>
>>>> -  if (!Parent && DIDescriptor(Scope).isSubprogram() &&
>>>> -      DISubprogram(Scope).describes(MF->getFunction()))
>>>> +  if (!Parent) {
>>>> +    assert(DIDescriptor(Scope).isSubprogram());
>>>> +    assert(DISubprogram(Scope).describes(MF->getFunction()));
>>>
>>>
>>> This assertion fails often while building Chromium on Windows.  I'll try to
>>> find a reduction.
>>
>> Thanks - looking forward to it. At least I seem to have made it out of
>> LLVM bootstraps/compiler-rt...
>
> Nope, spoke too soon.
>
> http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux-bootstrap/builds/3852/steps/build%20clang%2Fmsan/logs/stdio
>
> And I thought I'd done everything I could to reproduce this locally. :/