arsenm closed this revision.
arsenm added a comment.
r367973
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D65597/new/
https://reviews.llvm.org/D65597
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mai
rjmccall accepted this revision.
rjmccall added a comment.
This revision is now accepted and ready to land.
LGTM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D65597/new/
https://reviews.llvm.org/D65597
___
cfe-commits mailing list
cfe-commi
arsenm updated this revision to Diff 213276.
arsenm added a comment.
Change naming scheme, add more and tests
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D65597/new/
https://reviews.llvm.org/D65597
Files:
include/clang/Basic/Builtins.def
lib/CodeGen/CGBuiltin.cpp
test/CodeGenOpe
rjmccall added a comment.
In N2405 (http://www.open-std.org/jtc1/sc22/wg14/www/docs/n2405.pdf), these are
called `truncf16`, `cosf16`, etc., which I think is appropriate.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D65597/new/
https://reviews.llvm.org/D65597
___
arsenm created this revision.
arsenm added reviewers: rsmith, rjmccall, Anastasia, yaxunl.
Herald added a subscriber: wdng.
The implementation of the OpenCL builtin currently library uses 2 different
hacks to get to the corresponding IR intrinsics from the source. This will
allow removal of thos