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

Javed Absar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 1 13:01:16 PDT 2017


javed.absar added inline comments.


================
Comment at: test/CodeGen/neon-aapcs-align.c:9
+// RUN: -target-feature +neon -emit-llvm -w -o - %s | FileCheck %s -check-prefix=CHECK -check-prefix=OTHERS
+
+#include <arm_neon.h>
----------------
srhines wrote:
> Can you make sure that an Android triple doesn't trigger this? I worry because I am not sure what getABI() will return in that case.
Could you please elaborate.There is a test already  'armv7-none-linux-androideabi'.

lib/Basic/Targets.cpp:5357:: case llvm::Triple::Android:
sets it to aapcs-linux



https://reviews.llvm.org/D33786





More information about the llvm-commits mailing list