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