jlebar added a comment.

lg with one question about printf.


================
Comment at: lib/Headers/__clang_cuda_runtime_wrapper.h:237
@@ +236,3 @@
+// device-side declaration for it.
+__device__ int printf(const char *, ...);
+} // extern "C"
----------------
I think we want an attribute on this so that we know it's printf-like, so you 
get appropriate format-string warnings.  Unless the compiler is going to add 
said attribute automatically based on the just the function name.


http://reviews.llvm.org/D16638



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to