On 10/31/2016 09:45 AM, Andreas Schwab wrote: > On Okt 31 2016, Vineet Gupta <vineet.gup...@synopsys.com> wrote: > >> This silences tons of -Wformat= warnings when building ARC Linux kernel >> with gcc 6.x (and restores the ARC gcc 4.8.x behaviour) which had >> similar fix. >> >> gcc/ >> 2016-10-28 Vineet Gupta <vgu...@synopsys.com> >> >> * config/arc/arc.h (SIZE_TYPE): define as unsigned int. >> * (PTRDIFF_TYPE): define as int. >> >> Signed-off-by: Vineet Gupta <vgu...@synopsys.com> >> --- >> gcc/config/arc/arc.h | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/gcc/config/arc/arc.h b/gcc/config/arc/arc.h >> index cf13a7fa9fc3..0b579c44ee12 100644 >> --- a/gcc/config/arc/arc.h >> +++ b/gcc/config/arc/arc.h >> @@ -282,10 +282,10 @@ if (GET_MODE_CLASS (MODE) == MODE_INT \ >> #define DEFAULT_SIGNED_CHAR 0 >> >> #undef SIZE_TYPE >> -#define SIZE_TYPE "long unsigned int" >> +#define SIZE_TYPE "unsigned int" >> >> #undef PTRDIFF_TYPE >> -#define PTRDIFF_TYPE "long int" >> +#define PTRDIFF_TYPE "int" >> >> #undef WCHAR_TYPE >> #define WCHAR_TYPE "int" > This changes the C++ ABI. You need to fix the kernel instead, see > arch/avr32/include/uapi/asm/posix_types.h for how to do that.
There's ABI change one way or the other. I'd rather fix the upstream to have the ABI we want to continue with in the long run and match with the ABI we have in our off-upstream 4.8 and 6.x versions ! Thx, -Vineet
