yaxunl added inline comments. ================ Comment at: lib/Sema/SemaExpr.cpp:3431 @@ -3410,1 +3430,3 @@ + .getSupportedOpenCLOpts() + .cl_khr_fp64) || getOpenCLOptions().cl_khr_fp64)) { ---------------- 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. https://reviews.llvm.org/D24235 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits