[PATCH] D65000: [ARM] Set default alignment to 64bits

Mon Jul 22 02:08:29 PDT 2019

peter.smith added reviewers: srhines, danalbert.
peter.smith added a comment.

I think that this may not apply for Android as AFAIK their ABI still requires 128-bit alignment in some cases. Adding some more reviewers from 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))
----------------
I think that Android can require a higher alignment in some cases (See below). I think that this was explained in D33205

Repository:
  rC Clang

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

https://reviews.llvm.org/D65000