Sunil_Srivastava added a comment.
I had put these 'h' test lines to test the first of my changes, and this
sufficed, but you are correct. I should add %y too. I am going to add '%hr' and
'%hy'.
================
Comment at: test/Sema/format-strings-freebsd.c:39
@@ -35,1 +38,3 @@
+ freebsd_kernel_printf("%hr", h); // no-warning
+
// %y expects an int
----------------
dim wrote:
> 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.)
I had put these lines to test the first of my changes, and this sufficed, but
you are correct. I should add %y too. I am going to add '%hr' and '%hy'.
http://reviews.llvm.org/D19552
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
