[PATCH] D16692: [OpenCL] Eliminate warning when declaring OpenCL builtin functions

Xiuli PAN via cfe-commits cfe-commits at lists.llvm.org
Thu Jan 28 18:48:34 PST 2016


pxli168 added inline comments.

================
Comment at: lib/Sema/SemaLookup.cpp:698
@@ +697,3 @@
+        // tgmath.h, time.h, wchar.h and wctype.h are not available and cannot
+        // be included by a program.
+        if (S.getLangOpts().OpenCL &&
----------------
I think the  
> C99 standard headers
can cover those
> ...
>  // ctype.h, complex.h, errno.h, fenv.h, float.h, inttypes.h, limits.h,
>  // locale.h, setjmp.h, signal.h, stdarg.h, stdio.h, stdlib.h, string.h,
>  // tgmath.h, time.h, wchar.h and wctype.h
The comment shuold not be copy and paste as Anastasia will say.



http://reviews.llvm.org/D16692





More information about the cfe-commits mailing list