neil.hickey added inline comments.
================
Comment at: lib/Sema/SemaExpr.cpp:3431
@@ -3410,1 +3430,3 @@
+ .getSupportedOpenCLOpts()
+ .cl_khr_fp64) ||
getOpenCLOptions().cl_khr_fp64)) {
----------------
yaxunl wrote:
> This check
> (getLangOpts().OpenCLVersion >= 120 &&
> Context.getTargetInfo()
> .getSupportedOpenCLOpts()
> .cl_khr_fp64)
>
> is redundant since for CL 1.2 and above getOpenCLOptions().cl_khr_fp64 is set
> to be true by default.
This is get**Supported**OpenCLOpts(). Some hardware may not support doubles
https://reviews.llvm.org/D24235
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
