[PATCH] D127050: [Clang][FP16] Add 4 builtins for _Float16

[Phoebe Wang via Phabricator via cfe-commits]

pengfei added inline comments.


================
Comment at: clang/test/CodeGen/builtin_Float16.c:7
+void test_float16_builtins(void) {
+  volatile _Float16 res;
+
----------------
LuoYuanke wrote:
> Is _Float16 a legal type for target armv7a and aarch64?
Yes, see https://clang.llvm.org/docs/LanguageExtensions.html#half-precision-floating-point


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D127050