<div dir="ltr">Mystery solved. The failure was caused by another silly mistake -- missing test case changes in one commit. <div><br></div><div>About this change, what additional test do you recommend?</div><div><br></div><div>David</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, May 26, 2016 at 10:56 AM, Xinliang David Li <span dir="ltr"><<a href="mailto:davidxl@google.com" target="_blank">davidxl@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">The failed test case is the one  that actually covers this part of the code (there is also another one for darwin at LLVM side which does not fail) - so it does not look like it is due to random flakiness. Both bots are PPC (one LE and one BE).<span class="HOEnZb"><font color="#888888"><div><br></div><div>David</div></font></span></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Thu, May 26, 2016 at 10:50 AM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Nope - no idea, I haven't looked at it closely at all - just figured since it broke a buildbot (I assume you diagnosed that correctly - but do realize that many buildbots are flakey/complicated... so it may not be your fault if you're pretty sure this is No Functional Change)</div><div><div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, May 26, 2016 at 10:49 AM, Xinliang David Li <span dir="ltr"><<a href="mailto:davidxl@google.com" target="_blank">davidxl@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">I am a little puzzled why this change can make any change in compiler behavior. Any idea?<span><font color="#888888"><div><br></div><div>David</div></font></span></div><div><div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, May 26, 2016 at 10:38 AM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">(I see this was reverted - but when it goes back it it could use a test case)</div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, May 26, 2016 at 9:28 AM, Xinliang David Li via llvm-commits <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Author: davidxl<br>
Date: Thu May 26 11:28:01 2016<br>
New Revision: 270865<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=270865&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=270865&view=rev</a><br>
Log:<br>
Use new interface in Triple /NFC<br>
<br>
Modified:<br>
    llvm/trunk/lib/Transforms/Instrumentation/PGOInstrumentation.cpp<br>
<br>
Modified: llvm/trunk/lib/Transforms/Instrumentation/PGOInstrumentation.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/Instrumentation/PGOInstrumentation.cpp?rev=270865&r1=270864&r2=270865&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/Instrumentation/PGOInstrumentation.cpp?rev=270865&r1=270864&r2=270865&view=diff</a><br>
==============================================================================<br>
--- llvm/trunk/lib/Transforms/Instrumentation/PGOInstrumentation.cpp (original)<br>
+++ llvm/trunk/lib/Transforms/Instrumentation/PGOInstrumentation.cpp Thu May 26 11:28:01 2016<br>
@@ -790,7 +790,7 @@ static void createIRLevelProfileFlagVari<br>
       INSTR_PROF_QUOTE(IR_LEVEL_PROF_VERSION_VAR));<br>
   IRLevelVersionVariable->setVisibility(GlobalValue::DefaultVisibility);<br>
   Triple TT(M.getTargetTriple());<br>
-  if (TT.isOSBinFormatMachO())<br>
+  if (!TT.supportsCOMDAT())<br>
     IRLevelVersionVariable->setLinkage(GlobalValue::WeakAnyLinkage);<br>
   else<br>
     IRLevelVersionVariable->setComdat(M.getOrInsertComdat(<br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</blockquote></div><br></div>
</blockquote></div><br></div>
</div></div></blockquote></div><br></div>
</div></div></blockquote></div><br></div>
</div></div></blockquote></div><br></div>