[PATCH] D114099: Enable `_Float16` type support on X86 without the avx512fp16 flag

John McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Nov 18 08:37:04 PST 2021


rjmccall requested changes to this revision.
rjmccall added inline comments.
This revision now requires changes to proceed.


================
Comment at: clang/test/SemaCXX/Float16.cpp:4
+// RUN: %clang_cc1 -fsyntax-only -verify -triple armv7a-linux-gnu %s
+// RUN: %clang_cc1 -fsyntax-only -verify -triple aarch64-linux-gnu %s
 
----------------
This test (and Float16.c) should test at least one target that doesn't have `_Float16` support, so please just add `-DHAVE` to the x86_64 line and add, I dunno, a generic i386 or SPARC line.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D114099/new/

https://reviews.llvm.org/D114099



More information about the cfe-commits mailing list