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

[David Blaikie]

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. :/