<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Dec 15, 2017 at 7:22 AM, Evgeny Leviant <span dir="ltr"><<a href="mailto:eleviant@accesssoftek.com" target="_blank">eleviant@accesssoftek.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">This change causes following tests to fail:<br>
<br>
    LLVM :: LTO/Resolution/X86/common2.ll<br>
    LLVM :: LTO/Resolution/X86/dead-strip-<wbr>fulllto.ll<br>
    LLVM :: ThinLTO/X86/deadstrip.ll<br></blockquote><div><br></div><div>Multiple prevailing defs in these tests I suppose (looks like it in deadstrip.ll)? That's wrong, so good to fix these. Thanks, Teresa</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
I'll fix and commit.<br>
______________________________<wbr>__________<br>
От: Teresa Johnson via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>><br>
Отправлено: 15 декабря 2017 г. 17:52<br>
Кому: Evgeny Leviant; <a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>; <a href="mailto:tejohnson@google.com">tejohnson@google.com</a>; <a href="mailto:joker.eph@gmail.com">joker.eph@gmail.com</a>; <a href="mailto:peter@pcc.me.uk">peter@pcc.me.uk</a><br>
Копия: <a href="mailto:eraman@google.com">eraman@google.com</a>; <a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>; <a href="mailto:llvm.mail.list@gmail.com">llvm.mail.list@gmail.com</a>; George Rimar; Igor Kudrin<br>
Тема: [PATCH] D41113: [LLVMgold] Don't set resolutions for undefined symbols to 'Prevailing'<br>
<div class="HOEnZb"><div class="h5"><br>
tejohnson added inline comments.<br>
<br>
<br>
================<br>
Comment at: llvm/trunk/lib/LTO/LTO.cpp:420<br>
+    if (Res.Prevailing) {<br>
+      assert((GlobalRes.IRName.<wbr>empty() || !Sym.getIRName().empty()) &&<br>
+             "Overriding existing resolution with undefined asm symbol");<br>
----------------<br>
I had suggested changing this to simply:<br>
<br>
assert(GlobalRes.IRName.empty(<wbr>) && "....<br>
<br>
since we should only have one prevailing def. Can you do that?<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="https://reviews.llvm.org/D41113" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D41113</a><br>
<br>
<br>
<br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><span style="font-family:Times;font-size:medium"><table cellspacing="0" cellpadding="0"><tbody><tr style="color:rgb(85,85,85);font-family:sans-serif;font-size:small"><td nowrap style="border-top-style:solid;border-top-color:rgb(213,15,37);border-top-width:2px">Teresa Johnson |</td><td nowrap style="border-top-style:solid;border-top-color:rgb(51,105,232);border-top-width:2px"> Software Engineer |</td><td nowrap style="border-top-style:solid;border-top-color:rgb(0,153,57);border-top-width:2px"> <a href="mailto:tejohnson@google.com" target="_blank">tejohnson@google.com</a> |</td><td nowrap style="border-top-style:solid;border-top-color:rgb(238,178,17);border-top-width:2px"> 408-460-2413</td></tr></tbody></table></span></div>
</div></div>