[PATCH] Add flag to enable native half type

Pirama Arumuga Nainar pirama at google.com
Thu May 14 15:08:08 PDT 2015


A CC1 option is enough for our purposes, and matches the other fallow-half-* argument.


================
Comment at: test/CodeGen/fp16-ops.c:228-229
@@ -171,2 +227,4 @@
   // CHECK: fcmp ole
+  // NATIVE-HALF: fpext half
+  // NATIVE-HALF: fcmp ole float
   test = (h2 <= f0);
----------------
ab wrote:
> One thing you could do is separate all the clang-promoted comparisons like this and group them together.  You can then share CHECK- and F16TOF32 for these with HALF, if you add a NATIVE-HALF: [[F16TOF32:...]] and yet another, *really* shared CHECK prefix.  Not sure it helps though.
I'll leave this check as-is.  Having three different prefixes that are all interdependent is too complicated :)

http://reviews.llvm.org/D9781

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the cfe-commits mailing list