[PATCH] D122377: [PowerPC] Support 16-byte lock free atomics on pwr8 and up
Kai Luo via Phabricator via cfe-commits
cfe-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 cfe-commits
mailing list