[PATCH] D88546: [ARM] Update NEON testcase with missing target string in

Ranjeet Singh via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Sep 30 02:45:10 PDT 2020


rs created this revision.
rs added reviewers: SjoerdMeijer, john.brawn.
Herald added subscribers: cfe-commits, danielkiss, kristof.beyls.
Herald added a project: clang.
rs requested review of this revision.

This is a follow-up from https://reviews.llvm.org/D61717. Where Richard described the issue with compiling arm_neon.h under -flax-vector-conversions=none. It looks like the example reproducer does actually work:

echo '#include <arm_neon.h>' | clang -target arm64-linux-gnu -arch +neon -fsyntax-only -x c - -fno-lax-vector-conversions

but was missing the -ffreestanding compiler option otherwise the compiler would select the system headers from /usr. However, arm-neon-header.c test was missing this target triple 'arm64-linux-gnu' so I've added it here.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D88546

Files:
  clang/test/Headers/arm-neon-header.c


Index: clang/test/Headers/arm-neon-header.c
===================================================================
--- clang/test/Headers/arm-neon-header.c
+++ clang/test/Headers/arm-neon-header.c
@@ -22,5 +22,6 @@

 // RUN: %clang -fsyntax-only -Wall -Werror -ffreestanding --target=aarch64-none-eabi -march=armv8.2-a+fp16fml+crypto+dotprod -std=c11 -xc -flax-vector-conversions=none %s
 // RUN: %clang -fsyntax-only -Wall -Werror -ffreestanding --target=aarch64_be-none-eabi -march=armv8.2-a+fp16fml+crypto+dotprod -std=c11 -xc -flax-vector-conversions=none %s
+// RUN: %clang -fsyntax-only -Wall -Werror -ffreestanding --target=arm64-linux-gnu -arch +neon -std=c11 -xc -flax-vector-conversions=none %s

 #include <arm_neon.h>


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D88546.295219.patch
Type: text/x-patch
Size: 726 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20200930/50a8e090/attachment.bin>


More information about the cfe-commits mailing list