[PATCH] D50683: [Android] Set NewAlign for 64-bit Android to 8 bytes

Eli Friedman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Aug 15 14:13:04 PDT 2018


efriedma added a comment.

The C++ standard just says "An integer literal of type std::size_t whose value is the alignment guaranteed by a call to operator new(std::size_t) or operator new[](std::size_t)."  I would take that in the obvious sense, that any pointer returned by operator new must have alignment `__STDCPP_DEFAULT_NEW_ALIGNMENT__`.

Granted, I can see how that might not be the intent, given that `alignof(T) >= sizeof(T)` for all C++ types.


Repository:
  rC Clang

https://reviews.llvm.org/D50683





More information about the cfe-commits mailing list