<div dir="ltr"><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Jun 8, 2021 at 11:43 AM Joerg Sonnenberger via cfe-commits <<a href="mailto:cfe-commits@lists.llvm.org">cfe-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">On Mon, Jun 07, 2021 at 07:04:36PM -0700, via cfe-commits wrote:<br>
> <br>
> Author: =Jake Egan<br>
> Date: 2021-06-07T22:04:18-04:00<br>
> New Revision: e6629be31e67190f0a524f009752d73410894560<br>
> <br>
> URL: <a href="https://github.com/llvm/llvm-project/commit/e6629be31e67190f0a524f009752d73410894560" rel="noreferrer" target="_blank">https://github.com/llvm/llvm-project/commit/e6629be31e67190f0a524f009752d73410894560</a><br>
> DIFF: <a href="https://github.com/llvm/llvm-project/commit/e6629be31e67190f0a524f009752d73410894560.diff" rel="noreferrer" target="_blank">https://github.com/llvm/llvm-project/commit/e6629be31e67190f0a524f009752d73410894560.diff</a><br>
> <br>
> LOG: [AIX] Define __STDC_NO_ATOMICS__ and __STDC_NO_THREADS__ predefined macros<br>
<br>
I find the justification for this a bit strange. Neither one has to be<br>
implemented in libc. compiler-rt implements the former with the mixing<br>
caveats applied and there are portable implementations of the latter<br>
based on pthread. I would expect them to work well enough on AIX, too.<br>
So this seems to do more harm than good?<br></blockquote><div><br></div><div>Thanks for the feedback. The compiler-rt pointer is a good hint. Note that the status quo of compiler-rt is that the implementation you mention is guarded under<br></div><div style="margin-left:40px"><span style="font-family:monospace">if(APPLE)</span></div><div>in the CMake. Thus setting the macro reflects the status quo at this time.</div><div><br></div><div>For the latter, I think the vanilla LLVM source is meant to reflect what could be considered a vanilla deployment. The portable implementations you mention would be an extra component.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Joerg<br>
_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a><br>
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits</a><br>
</blockquote></div></div>