Message ID | 5193758A.7020103@ubuntu.com |
---|---|
State | New |
Headers | show |
ping 2 Am 15.05.2013 13:46, schrieb Matthias Klose: > ping? > > regenerated the patch for the trunk, check with builds on arm-linux-gnueabihf > and x86_64-linux-gnu > > Matthias > > * libgcc2.c: Don't include <limits.h>. > > Am 14.01.2013 22:54, schrieb Matthias Klose: >> Am 04.01.2013 20:01, schrieb Wookey: >>> I filed http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55743 (my first >>> upstream gcc bug so be gentle :-) >>> >>> Details are there but the short version is that the limits.h inclusion >>> in libgcc2.c is now a relic because the constants that it brings >>> in are no longer used (since >>> http://repo.or.cz/w/official-gcc.git/blobdiff/49f0f270673c4512c11f72a038b84c321ae5534a..7429c938827aa98bf3b02c4ac89510f4d28ef0b1:/gcc/libgcc2.c >>> ) >>> >>> And this inclusion can break --without-headers bootstrapping (which is >>> how I noticed it). >>> >>> Doko poked me to send the patch to this list for consideration for >>> inclusion in trunk. >> >> The --without-headers build failures is unrelated. To catch this >> mis-configuration I did propose a patch in >> http://gcc.gnu.org/ml/gcc-patches/2013-01/msg00743.html >> >> I think the patch itself is correct. However >> >> - please submit your patch against trunk, and state that you >> did test the patch against trunk (of course, after testing it) >> >> - please provide a ChangeLog entry >> >> - thanks for your reference to the repo.or.cz repo, however it >> would be good to reference a GCC commit. >> looks like Alexandre Oliva did commit this without removing >> the unneeded bits in r39365. >> >> Matthias >> >> >
This patch is OK. Sorry for not looking at it earlier. Thanks. Ian On Mon, Jul 15, 2013 at 5:40 AM, Matthias Klose <doko@ubuntu.com> wrote: > ping 2 > > Am 15.05.2013 13:46, schrieb Matthias Klose: >> ping? >> >> regenerated the patch for the trunk, check with builds on arm-linux-gnueabihf >> and x86_64-linux-gnu >> >> Matthias >> >> * libgcc2.c: Don't include <limits.h>. >> >> Am 14.01.2013 22:54, schrieb Matthias Klose: >>> Am 04.01.2013 20:01, schrieb Wookey: >>>> I filed http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55743 (my first >>>> upstream gcc bug so be gentle :-) >>>> >>>> Details are there but the short version is that the limits.h inclusion >>>> in libgcc2.c is now a relic because the constants that it brings >>>> in are no longer used (since >>>> http://repo.or.cz/w/official-gcc.git/blobdiff/49f0f270673c4512c11f72a038b84c321ae5534a..7429c938827aa98bf3b02c4ac89510f4d28ef0b1:/gcc/libgcc2.c >>>> ) >>>> >>>> And this inclusion can break --without-headers bootstrapping (which is >>>> how I noticed it). >>>> >>>> Doko poked me to send the patch to this list for consideration for >>>> inclusion in trunk. >>> >>> The --without-headers build failures is unrelated. To catch this >>> mis-configuration I did propose a patch in >>> http://gcc.gnu.org/ml/gcc-patches/2013-01/msg00743.html >>> >>> I think the patch itself is correct. However >>> >>> - please submit your patch against trunk, and state that you >>> did test the patch against trunk (of course, after testing it) >>> >>> - please provide a ChangeLog entry >>> >>> - thanks for your reference to the repo.or.cz repo, however it >>> would be good to reference a GCC commit. >>> looks like Alexandre Oliva did commit this without removing >>> the unneeded bits in r39365. >>> >>> Matthias >>> >>> >> >
Index: libgcc/libgcc2.c =================================================================== --- libgcc/libgcc2.c (revision 198927) +++ libgcc/libgcc2.c (working copy) @@ -1674,18 +1674,6 @@ #endif #if defined(L_fixunsxfsi) && LIBGCC2_HAS_XF_MODE -/* Reenable the normal types, in case limits.h needs them. */ -#undef char -#undef short -#undef int -#undef long -#undef unsigned -#undef float -#undef double -#undef MIN -#undef MAX -#include <limits.h> - UWtype __fixunsxfSI (XFtype a) { @@ -1696,18 +1684,6 @@ #endif #if defined(L_fixunsdfsi) && LIBGCC2_HAS_DF_MODE -/* Reenable the normal types, in case limits.h needs them. */ -#undef char -#undef short -#undef int -#undef long -#undef unsigned -#undef float -#undef double -#undef MIN -#undef MAX -#include <limits.h> - UWtype __fixunsdfSI (DFtype a) { @@ -1718,18 +1694,6 @@ #endif #if defined(L_fixunssfsi) && LIBGCC2_HAS_SF_MODE -/* Reenable the normal types, in case limits.h needs them. */ -#undef char -#undef short -#undef int -#undef long -#undef unsigned -#undef float -#undef double -#undef MIN -#undef MAX -#include <limits.h> - UWtype __fixunssfSI (SFtype a) {