[PATCH] D52412: OpenCL: Mark printf format string argument

2018-10-06 Thread James Price via Phabricator via cfe-commits
jprice added a comment. This change has caused Clang to start emitting erroneous warnings when using OpenCL's vector specifier, e.g. "%v4f", which doesn't seem to be covered by the format string analysis. https://reviews.llvm.org/D52412 ___ cfe-co

[PATCH] D52412: OpenCL: Mark printf format string argument

2018-10-02 Thread Matt Arsenault via Phabricator via cfe-commits
arsenm closed this revision. arsenm added a comment. r343653 https://reviews.llvm.org/D52412 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D52412: OpenCL: Mark printf format string argument

2018-10-02 Thread Anastasia Stulova via Phabricator via cfe-commits
Anastasia accepted this revision. Anastasia added a comment. This revision is now accepted and ready to land. LGTM! Thanks! Sorry I missed this somehow. https://reviews.llvm.org/D52412 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://l

[PATCH] D52412: OpenCL: Mark printf format string argument

2018-10-01 Thread Matt Arsenault via Phabricator via cfe-commits
arsenm added a comment. ping https://reviews.llvm.org/D52412 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D52412: OpenCL: Mark printf format string argument

2018-09-24 Thread Matt Arsenault via Phabricator via cfe-commits
arsenm created this revision. arsenm added a reviewer: Anastasia. Herald added subscribers: yaxunl, wdng. Fixes not warning on format string errors. https://reviews.llvm.org/D52412 Files: lib/Headers/opencl-c.h test/SemaOpenCL/printf-format-string-warnings.cl Index: test/SemaOpenCL/printf