[PATCH] D127189: [clang][AIX] Add option to control quadword lock free atomics ABI on AIX

Tue Jun 28 20:38:30 PDT 2022

lkail added inline comments.

================
Comment at: clang/test/Driver/ppc-unsupported.c:12
+// RUN:   -c %s 2>&1 | FileCheck %s
+// RUN: not %clang -target powerpc64le-unknown-linux -maix64-quadword-atomics \
+// RUN:   -c %s 2>&1 | FileCheck %s
----------------
amyk wrote:
> amyk wrote:
> > Should we have a big endian Linux check, too?
> Oh, sorry. I noticed there wasn't `powerpc64-unknown-linux` but I realized I think `powerpc64-unknown-freebsd` is supposed to be the big endian 64-bit Linux check, right?
Added OS check is following lines above. I'm ok to add `powerpc64-unknown-linux` too. Thanks for pointing it out.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D127189