[PATCH] D33719: Add _Float16 as a C/C++ source language type

[Roger Ferrer Ibanez via Phabricator via cfe-commits]

rogfer01 added a comment.

There are a few tests checking the precise interaction between `__fp16` and `_Float16` values but I don't see tests interacting `_Float16` and the other standard floating types `float`, `double` and `long double`.

Do you reckon it is worth adding them as well?



================
Comment at: test/Frontend/float16.cpp:82
+
+// When do have native half types, we expect to see promotions XXXXX:
+
----------------
Is this XXXXX some form of TODO?


https://reviews.llvm.org/D33719