<div dir="ltr"><div dir="ltr"><a class="gmail_plusreply" id="plusReplyChip-0" href="mailto:i@maskray.me" tabindex="-1">@Fangrui Song</a><div>, multiple PPC bots are failing on clang/test/OpenMP/parallel_codegen.cpp after this commit:</div><div><br></div><div>
<div><a href="http://lab.llvm.org:8011/#/builders/52/builds/2939">http://lab.llvm.org:8011/#/builders/52/builds/2939</a></div>

<a href="http://lab.llvm.org:8011/#/builders/57/builds/2938">http://lab.llvm.org:8011/#/builders/57/builds/2938</a></div><div><br></div><div>These bots were passing immediately before your commits.</div><div><br></div><div><span style="font-family:monospace">> git log --oneline 14dc69b09218bb580069eb547bb16ab0c43db6d4..d1fd72343c6ff58a3b66bc0df56fed9ac21e4056<br>d1fd723 Refactor how -fno-semantic-interposition sets dso_local on default visibility external linkage definitions<br>219d00e [test] Make ELF tests immune to dso_local/dso_preemptable/(none) differences</span><br></div></div><div><br></div><div>Can you have a look?</div><div><br></div><div>-- HT<br></div><div><br></div><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Dec 31, 2020 at 5:00 PM Fangrui Song via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.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"><br>
Author: Fangrui Song<br>
Date: 2020-12-31T13:59:45-08:00<br>
New Revision: d1fd72343c6ff58a3b66bc0df56fed9ac21e4056<br>
<br>
URL: <a href="https://github.com/llvm/llvm-project/commit/d1fd72343c6ff58a3b66bc0df56fed9ac21e4056" rel="noreferrer" target="_blank">https://github.com/llvm/llvm-project/commit/d1fd72343c6ff58a3b66bc0df56fed9ac21e4056</a><br>
DIFF: <a href="https://github.com/llvm/llvm-project/commit/d1fd72343c6ff58a3b66bc0df56fed9ac21e4056.diff" rel="noreferrer" target="_blank">https://github.com/llvm/llvm-project/commit/d1fd72343c6ff58a3b66bc0df56fed9ac21e4056.diff</a><br>
<br>
LOG: Refactor how -fno-semantic-interposition sets dso_local on default visibility external linkage definitions<br>
<br>
The idea is that the CC1 default for ELF should set dso_local on default<br>
visibility external linkage definitions in the default -mrelocation-model pic<br>
mode (-fpic/-fPIC) to match COFF/Mach-O and make output IR similar.<br>
<br>
The refactoring is made available by 2820a2ca3a0e69c3f301845420e0067ffff2251b.<br>
<br>
Currently only x86 supports local aliases. We move the decision to the driver.<br>
There are three CC1 states:<br>
<br>
* -fsemantic-interposition: make some linkages interposable and make default visibility external linkage definitions dso_preemptable.<br>
* (default): selected if the target supports .Lfoo$local: make default visibility external linkage definitions dso_local<br>
* -fhalf-no-semantic-interposition: if neither option is set or the target does not support .Lfoo$local: like -fno-semantic-interposition but local aliases are not used. So references can be interposed if not optimized out.<br>
<br>
Add -fhalf-no-semantic-interposition to a few tests using the half-based semantic interposition behavior.<br>
</blockquote></div></div>