SjoerdMeijer added inline comments.
================
Comment at: test/Sema/aarch64-neon-fp16-ranges.c:1
+// RUN: %clang_cc1 -triple arm64-linux-gnu -target-feature +neon
-fallow-half-arguments-and-returns -target-feature +fullfp16 -ffreestanding
-fsyntax-only -verify %s
+// RUN: %clang_cc1 -triple aarch64-linux-gnu -target-feature +neon
-fallow-half-arguments-and-returns -target-feature +fullfp16 -ffreestanding
-fsyntax-only -verify %s
----------------
Nit: target feature fullfp16 implies ARMv8 FP, so I think you can remove +neon;
just a tiny optimisation to make the command line shorter (same below).
================
Comment at: test/Sema/aarch64-neon-fp16-ranges.c:39
+
+void test_vcvt_su_f(int64_t a){
+ vcvth_n_s16_f16(a, 1);
----------------
why is this is 'a' an int64_t? Should this not be float16_t?
https://reviews.llvm.org/D47592
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
