[PATCH] D122377: [PowerPC] Support 16-byte lock free atomics on pwr8 and up

Kai Luo via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 5 19:00:09 PDT 2022


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



More information about the llvm-commits mailing list