On Jan 17, 2013, at 14:11 , Jordan Rose <[email protected]> wrote:
> > On Jan 17, 2013, at 13:04 , Dmitri Gribenko <[email protected]> wrote: > >> Hi Jordan, >> >> On Thu, Jan 17, 2013 at 8:47 PM, Jordan Rose <[email protected]> wrote: >>> Author: jrose >>> Date: Thu Jan 17 12:47:16 2013 >>> New Revision: 172739 >>> >>> URL: http://llvm.org/viewvc/llvm-project?rev=172739&view=rev >>> Log: >>> Format strings: correct signedness if already correcting width (%d,%u). >>> >>> It is valid to do this: >>> printf("%u", (int)x); >>> >>> But if we see this: >>> printf("%lu", (int)x); >>> >>> ...our fixit should suggest %d, not %u. >> >> Sorry, but it looks like this commit broke Linux/x86-64: >> >> http://lab.llvm.org:8011/builders/clang-x86_64-debian-fast/builds/422 > > > I don't know why my check didn't catch these; I'm seeing them locally now > too. Thanks, Dmitri. Hopefully fixed in r172762! _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
