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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
