dim accepted this revision.
dim added a comment.
This revision is now accepted and ready to land.

LGTM.


================
Comment at: test/Sema/format-strings-freebsd.c:39
@@ -35,1 +38,3 @@
+  freebsd_kernel_printf("%hr", h); // no-warning
+
   // %y expects an int
----------------
It's nice to add these for %r, but is there any reason you didn't add similar 
ones for %y? (This is just a minor nit.)


http://reviews.llvm.org/D19552



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

Reply via email to