[PATCH] Remove unnecessaily included limits.h in libgcc2.c

[PATCH] Remove unnecessaily included limits.h in libgcc2.c

[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.

--- gcc-4.7-4.7.2/src/libgcc/libgcc2.c.orig	2011-11-02 15:26:35.000000000 +0000
+++ gcc-4.7-4.7.2/src/libgcc/libgcc2.c	2012-12-18 19:33:40.000000000 +0000
@@ -1676,18 +1676,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)
 {
@@ -1698,18 +1686,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)
 {
@@ -1720,18 +1696,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)
 {


Wookey
-- 
Principal hats:  Linaro, Emdebian, Wookware, Balloonboard, ARM
http://wookware.org/

Re: [PATCH] Remove unnecessaily included limits.h in libgcc2.c

[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

[ping] Re: [PATCH] Remove unnecessaily included limits.h in libgcc2.c

[Matthias Klose]

[-- Attachment #1: Type: text/plain, Size: 1514 bytes --]

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
> 
> 


[-- Attachment #2: libgcc.diff --]
[-- Type: text/plain, Size: 1151 bytes --]

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)
 {

[ping 2] Re: [ping] Re: [PATCH] Remove unnecessaily included limits.h in libgcc2.c

[Matthias Klose]

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
>>
>>
> 

Re: [ping 2] Re: [ping] Re: [PATCH] Remove unnecessaily included limits.h in libgcc2.c

[Ian Lance Taylor]

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
>>>
>>>
>>
>