<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-family:monospace;font-size:small;color:#000000"><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Jun 11, 2020 at 12:37 AM Petr Hosek via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">phosek added a comment.<br>
<br>
In D76802#2086757 <<a href="https://reviews.llvm.org/D76802#2086757" rel="noreferrer" target="_blank">https://reviews.llvm.org/D76802#2086757</a>>, @vitalybuka wrote:<br>
<br>
> In D76802#2086742 <<a href="https://reviews.llvm.org/D76802#2086742" rel="noreferrer" target="_blank">https://reviews.llvm.org/D76802#2086742</a>>, @phosek wrote:<br>
><br>
> > In D76802#2086725 <<a href="https://reviews.llvm.org/D76802#2086725" rel="noreferrer" target="_blank">https://reviews.llvm.org/D76802#2086725</a>>, @vitalybuka wrote:<br>
> ><br>
> > > In D76802#2086650 <<a href="https://reviews.llvm.org/D76802#2086650" rel="noreferrer" target="_blank">https://reviews.llvm.org/D76802#2086650</a>>, @phosek wrote:<br>
> > ><br>
> > > > I looked into these issue:<br>
> > > ><br>
> > > > - <a href="http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux/builds/27806" rel="noreferrer" target="_blank">http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux/builds/27806</a> failure was addressed by rG69c5ff4668cd4d1bfb6b45d71e15c12e3a23ae05 <<a href="https://reviews.llvm.org/rG69c5ff4668cd4d1bfb6b45d71e15c12e3a23ae05" rel="noreferrer" target="_blank">https://reviews.llvm.org/rG69c5ff4668cd4d1bfb6b45d71e15c12e3a23ae05</a>><br>
> > ><br>
> > ><br>
> > > 69c5ff4668cd4d1bfb6b45d71e15c12e3a23ae05 <<a href="https://reviews.llvm.org/rG69c5ff4668cd4d1bfb6b45d71e15c12e3a23ae05" rel="noreferrer" target="_blank">https://reviews.llvm.org/rG69c5ff4668cd4d1bfb6b45d71e15c12e3a23ae05</a>> still fails "check-profile" for me and fails here <a href="http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux/builds/27808" rel="noreferrer" target="_blank">http://lab.llvm.org:8011/builders/sanitizer-x86_64-linux/builds/27808</a><br>
> ><br>
> ><br>
> > I've updated the patch to handle the `_IO_stdin_used` symbol in the test. I couldn't reproduce this locally, but it's possible that this is due to different libc version on the bot.<br>
><br>
><br>
> I still can reproduce with the latest patch on my workstation<br>
><br>
>   Exit Code: 1<br>
>  <br>
>   Command Output (stderr):<br>
>   --<br>
>   /usr/bin/ld: __llvm_prf_data has both ordered and unordered sections<br>
>   /usr/bin/ld: final link failed: bad value<br>
>   clang-11: error: linker command failed with exit code 1 (use -v to see invocation)<br>
><br>
<br>
<br>
This is the other issue I mentioned which also affects Chromium bots. The root cause is the lack of support for the extended semantics of `SHF_LINK_ORDER` that `!associated` metadata rely on in bfd.ld. I'm trying to figure out what's the best way forward. We could just disable all affected profile tests when bfd.ld is used as the linker, but that's pretty drastic.</blockquote><div><br></div><div class="gmail_default" style="font-family:monospace;font-size:small;color:rgb(0,0,0)">This won't work well as those who use bfd.ld won't be able to use PGO anymore.</div><div class="gmail_default" style="font-family:monospace;font-size:small;color:rgb(0,0,0)"></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"> Another option would be to gate this feature on a backend flag; targets that use gold or lld as their linker could turn this on by default.<br></blockquote><div><br></div><div class="gmail_default" style="font-family:monospace;font-size:small;color:rgb(0,0,0)">this might be the way to go.</div><div class="gmail_default" style="font-family:monospace;font-size:small;color:rgb(0,0,0)"><br></div><div class="gmail_default" style="font-family:monospace;font-size:small;color:rgb(0,0,0)">David</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
<br>
CHANGES SINCE LAST ACTION<br>
  <a href="https://reviews.llvm.org/D76802/new/" rel="noreferrer" target="_blank">https://reviews.llvm.org/D76802/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D76802" rel="noreferrer" target="_blank">https://reviews.llvm.org/D76802</a><br>
<br>
<br>
<br>
</blockquote></div></div>