This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: fix some printf mismatches


On 5/18/07, Geoffrey Keating <gkeating@apple.com> wrote:

Rafael's patch caused GCC to start warning about using %x with signed values. Although the patch had other problems, this warning in itself is not wrong; ISO C doesn't strictly permit this (although it is not a problem on any platform I know about). It turns out that looking for places where people have been sloppy about this is pretty good at finding other problems, as in the sched-vis change below, so I went through and fixed them.

Can you wait a second, these warnings are wrong. This is what exactly why the patch was incorrect in the first place. The original patch still have other issues, see my email about the regressions on powerpc-linux-gnu/spu-elf.


-- Pinski



Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]