[PATCH] D37231: Add half load and store builtins

Anastasia Stulova via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Sep 6 10:43:27 PDT 2017


Anastasia added inline comments.


================
Comment at: test/CodeGenOpenCL/no-half.cl:27
+	foo[0] = __builtin_load_halff(bar);
+// CHECK: [[HALF_VAL:%.*]] = load
+// CHECK: [[FULL_VAL:%.*]] = fpext half [[HALF_VAL]] to float
----------------
jvesely wrote:
> Anastasia wrote:
> > Minor thing: any reason you are not checking the load fully?
> just my laziness, I've added full check.
Could we do the same for the above examples too?


Repository:
  rL LLVM

https://reviews.llvm.org/D37231





More information about the cfe-commits mailing list