[PATCH] D33205: ARM] Fix Neon vector type alignment to 64-bit

[Renato Golin via Phabricator via llvm-commits]

rengolin added inline comments.


================
Comment at: lib/Basic/Targets.cpp:5300
 
+    MaxVectorAlign = 64; // 64-bit alignment (AAPCS)
+
----------------
srhines wrote:
> This should not be set for Android targets. Doing so would break our existing ABI.
Regardless of the merits, being practical, I think we should make this condition to the environment, to make sure that at least Android keeps its original alignment.

Docs updates and future changes to the ABI should be handled separately.


https://reviews.llvm.org/D33205