<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Dec 16, 2014 at 2:55 PM, Duncan P. N. Exon Smith <span dir="ltr"><<a href="mailto:dexonsmith@apple.com" target="_blank">dexonsmith@apple.com</a>></span> wrote:<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class=""><br>
> On 2014 Dec 16, at 14:38, Rafael Espíndola <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> wrote:<br>
><br>
>> It's asymmetrical (asymmetry happens all the time in linkers, right?), but<br>
>> I'm not sure there's a good way to fix this without invasive changes to<br>
>> `MapValue()`.  I'd rather leave "solving this correctly" until we have a<br>
>> custom class for subprograms (as mentioned in PR21910).<br>
><br>
> So the logic is asymmetrical, but the end result is normally<br>
> symmetrical, which is why some tests use<br>
> ; RUN: llvm-link %p/foo.ll %p/Inputs/foo.ll ...<br>
> ; RUN: llvm-link %p/Inputs/foo.ll %p/foo.ll ...<br>
><br>
> But yes, this can be done in another patch. Please just leave a FIXME<br>
> in the code somewhere.<br>
<br>
</span>Done.<br>
<span class=""><br>
>> (I wouldn't even be trying to solve this yet if it weren't blocking<br>
>> our Release+Asserts bootstrap.)<br>
><br>
> I think the patch is OK with the SrcM->getNamedMetadata("<a href="http://llvm.dbg.cu" target="_blank">llvm.dbg.cu</a>")<br>
> check removed.<br>
<br>
</span>Attached.<br>
<br>
@David, does this look okay to you too?  (I won't be closing PR21910; I<br>
figure we can use that to track a better long-term fix.)<br></blockquote><div><br>Sure thing<br> </div></div></div></div>