[PATCH] D19552: Check 'r' and 'y specifiers of freebsd_kernel_printf format strings on PS4
Dimitry Andric via cfe-commits
cfe-commits at lists.llvm.org
Tue Apr 26 13:21:26 PDT 2016
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
More information about the cfe-commits
mailing list