Re: [PATCH] allow -Werror on IA-64 with GPT enabled

[Andrew Clausen]

On Fri, Oct 05, 2001 at 03:37:56PM -0500, address@hidden wrote:
> I've posted a patch against 1.4.20-pre2, which already includes the GPT
> patch (thanks!) to set printf'ing of 64-bit values based on the platform
> type.  Currently, on IA-64, with the -Werror flag passed to gcc, the build
> fails because %llx doesn't work for printing 64-bit values (even though long
> long is 64 bits on both x86 and IA-64).  With this patch, you no longer have
> to --disable-Werror when configuring on IA-64.

How painful! :(

What does #ifdef __LP64__ mean?  (my rather old gcc and libc info pages
don't mention it)  From looking on gcc archives, it means "32 bit int,
64 bit longs and pointers")

        http://gcc.gnu.org/ml/gcc/2000-03/msg00749.html

Shouldn't we be checking for something like #if sizeof(int) == 8 instead?

Thanks,
Andrew