[PATCH] D88659: [FE]Split SuitableAlign into two parts

Hubert Tong via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Oct 8 09:24:34 PDT 2020


hubert.reinterpretcast added a comment.

In D88659#2318203 <https://reviews.llvm.org/D88659#2318203>, @jyknight wrote:

> It seems like on AIX, `__BIGGEST_ALIGNMENT__` should just be set to 16, then. I'm not sure why you want it to be 8?



  /// Return the alignment that is suitable for storing any
  /// object with a fundamental alignment requirement.

Vector types have extended (not fundamental) alignment on AIX, because `malloc` is not guaranteed to return addresses that are 16-byte aligned.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D88659



More information about the cfe-commits mailing list