[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