lkail added a comment.

In D122377#3428533 <https://reviews.llvm.org/D122377#3428533>, 
@hubert.reinterpretcast wrote:

> I am not sure that the choice of `isOSBinFormatELF` to (afaik) primarily 
> scope this change from affecting AIX (where we know the library calls are not 
> implemented to be lock-free yet) is better than alternative where the 
> condition is for little-endian mode or specifically for not AIX.

I agree with we should exclude AIX explicitly(use `!isOSAIX()` rather than 
`isOSBinFormatELF()`) for current situation where AIX's support is on progress 
as we have known. What is unknown at present is the impact on freebsd.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D122377/new/

https://reviews.llvm.org/D122377

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to