hubert.reinterpretcast added inline comments.
================
Comment at: clang/test/CodeGen/PowerPC/atomic-alignment.c:45
+// PPC64: @q = global %struct.Q zeroinitializer, align 16{{$}}
+// AIX64: @q = global %struct.Q zeroinitializer, align 1{{$}}
_Atomic(Q) q; // expected-no-diagnostics
----------------
Although it is a change that affects layout compatibility with the short
history of 16-byte atomic types compiled with LLVM-based compilers on AIX,
having this 16-byte atomic type aligned also on AIX for PPC64 would be
reasonable. The consequences of not doing so is that operations on some types
would be lock-free only on ELF-based platforms.
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
