[PATCH] D146425: [SystemZ] Enable AtomicExpandPass for i128

Ulrich Weigand via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 31 09:24:44 PDT 2023


uweigand added a subscriber: Andreas-Krebbel.
uweigand added a comment.

@Andreas-Krebbel any comments on this discrepancy between C and C++ in GCC?   Was this intentional?

> Both clang++ and g++ aligns this to 16 bytes:
> `std::atomic<__int128> Atomic_int128;         // C++`
>
> However, while clang aligns this also to 16 bytes, GCC aligns only to 8, which I suspect is an error by GCC(?):
> `Atomic __int128 Atomic_int128;               // C`




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

https://reviews.llvm.org/D146425



More information about the llvm-commits mailing list