[PATCH] D37231: Add half load and store builtins

[Jan Vesely via Phabricator via cfe-commits]

jvesely marked 2 inline comments as done.
jvesely 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
----------------
Anastasia wrote:
> 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?
I don't understand.
if you mean test_store_*, those functions do not generate any load instructions. the full generated code is:
test_store_double:
```
entry:
  %0 = fptrunc double %foo to half
  store half %0, half addrspace(1)* %bar, align 2
  ret void
```


Repository:
  rL LLVM

https://reviews.llvm.org/D37231