[PATCH] D65000: [ARM] Set default alignment to 64bits
Diogo N. Sampaio via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Jul 22 03:10:25 PDT 2019
dnsampaio marked 2 inline comments as done.
dnsampaio added a comment.
Set android-abi default to 128. Added tests for android and not-android.
================
Comment at: lib/Basic/Targets/ARM.cpp:311
// Maximum alignment for ARM NEON data types should be 64-bits (AAPCS)
if (IsAAPCS && (Triple.getEnvironment() != llvm::Triple::Android))
----------------
peter.smith wrote:
> I think that Android can require a higher alignment in some cases (See below). I think that this was explained in D33205
Hi @peter.smith ,
thanks for the review, I did not know about the androidabi being 128 bits default. Updated accordingly.
Repository:
rC Clang
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D65000/new/
https://reviews.llvm.org/D65000
More information about the cfe-commits
mailing list