[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
Tue Aug 14 11:11:40 PDT 2018
efriedma accepted this revision.
efriedma added a comment.
This revision is now accepted and ready to land.
LGTM
================
Comment at: lib/Basic/TargetInfo.cpp:72
+ // For 64-bit Android, alignment is 8 bytes for allocations <= 8 bytes.
+ NewAlign = (Triple.isArch64Bit() || Triple.isArch32Bit()) ? 64 : 0;
+ } else
----------------
Might as well just set `NewAlign = 64;` here. But not a big deal either way.
Repository:
rC Clang
https://reviews.llvm.org/D50683
More information about the cfe-commits
mailing list