Patchwork [print-tree.c] : 5 of 7 Fix of PR target/53912 bootstrap fails using default c++ mode in stage 2 and 3 for native x86_64-w64-mingw32

login
register
mail settings
Submitter Kai Tietz
Date Nov. 29, 2012, 12:11 p.m.
Message ID <CAEwic4bgUKnNx_JJM4Hp2kupA+=OwX3sSWAk=i3ouw2ByBtUBg@mail.gmail.com>
Download mbox | patch
Permalink /patch/202726/
State New
Headers show

Comments

Kai Tietz - Nov. 29, 2012, 12:11 p.m.
Hello,

this trivial patch fixes a bootstrap issue on LLP64 hosts.

ChangeLog

2012-11-29 Kai Tietz

	PR target/53912
	* print-tree.c (print_node): Cast from pointer via uintptr_t.

Tested for i686-w64-mingw32, x86_64-w64-mingw32, and
x86_64-unknown-gnu-linux. Ok for apply?

Regards,
Kai
Kai Tietz - Dec. 2, 2012, 9:12 a.m.
Ping

2012/11/29 Kai Tietz <ktietz70@googlemail.com>:
> Hello,
>
> this trivial patch fixes a bootstrap issue on LLP64 hosts.
>
> ChangeLog
>
> 2012-11-29 Kai Tietz
>
>         PR target/53912
>         * print-tree.c (print_node): Cast from pointer via uintptr_t.
>
> Tested for i686-w64-mingw32, x86_64-w64-mingw32, and
> x86_64-unknown-gnu-linux. Ok for apply?
>
> Regards,
> Kai
>
> Index: print-tree.c
> ===================================================================
> --- print-tree.c        (Revision 193925)
> +++ print-tree.c        (Arbeitskopie)
> @@ -255,7 +255,7 @@ print_node (FILE *file, const char *prefix, tree n
>    /* Allow this function to be called if the table is not there.  */
>    if (table)
>      {
> -      hash = ((unsigned long) node) % HASH_SIZE;
> +      hash = ((uintptr_t) node) % HASH_SIZE;
>
>        /* If node is in the table, just mention its address.  */
>        for (b = table[hash]; b; b = b->next)
NightStrike - Dec. 9, 2012, 4:18 p.m.
On Sat, Dec 1, 2012 at 11:12 PM, Kai Tietz <ktietz70@googlemail.com> wrote:
> Ping
>
> 2012/11/29 Kai Tietz <ktietz70@googlemail.com>:
>> Hello,
>>
>> this trivial patch fixes a bootstrap issue on LLP64 hosts.
>>
>> ChangeLog
>>
>> 2012-11-29 Kai Tietz
>>
>>         PR target/53912
>>         * print-tree.c (print_node): Cast from pointer via uintptr_t.
>>
>> Tested for i686-w64-mingw32, x86_64-w64-mingw32, and
>> x86_64-unknown-gnu-linux. Ok for apply?
>>
>> Regards,
>> Kai
>>
>> Index: print-tree.c
>> ===================================================================
>> --- print-tree.c        (Revision 193925)
>> +++ print-tree.c        (Arbeitskopie)
>> @@ -255,7 +255,7 @@ print_node (FILE *file, const char *prefix, tree n
>>    /* Allow this function to be called if the table is not there.  */
>>    if (table)
>>      {
>> -      hash = ((unsigned long) node) % HASH_SIZE;
>> +      hash = ((uintptr_t) node) % HASH_SIZE;
>>
>>        /* If node is in the table, just mention its address.  */
>>        for (b = table[hash]; b; b = b->next)

Ping.
Richard Guenther - Dec. 10, 2012, 9:21 a.m.
On Sun, Dec 9, 2012 at 5:18 PM, NightStrike <nightstrike@gmail.com> wrote:
> On Sat, Dec 1, 2012 at 11:12 PM, Kai Tietz <ktietz70@googlemail.com> wrote:
>> Ping
>>
>> 2012/11/29 Kai Tietz <ktietz70@googlemail.com>:
>>> Hello,
>>>
>>> this trivial patch fixes a bootstrap issue on LLP64 hosts.
>>>
>>> ChangeLog
>>>
>>> 2012-11-29 Kai Tietz
>>>
>>>         PR target/53912
>>>         * print-tree.c (print_node): Cast from pointer via uintptr_t.
>>>
>>> Tested for i686-w64-mingw32, x86_64-w64-mingw32, and
>>> x86_64-unknown-gnu-linux. Ok for apply?
>>>
>>> Regards,
>>> Kai
>>>
>>> Index: print-tree.c
>>> ===================================================================
>>> --- print-tree.c        (Revision 193925)
>>> +++ print-tree.c        (Arbeitskopie)
>>> @@ -255,7 +255,7 @@ print_node (FILE *file, const char *prefix, tree n
>>>    /* Allow this function to be called if the table is not there.  */
>>>    if (table)
>>>      {
>>> -      hash = ((unsigned long) node) % HASH_SIZE;
>>> +      hash = ((uintptr_t) node) % HASH_SIZE;
>>>
>>>        /* If node is in the table, just mention its address.  */
>>>        for (b = table[hash]; b; b = b->next)
>
> Ping.

Ok.

Thanks,
Richard.

Patch

Index: print-tree.c
===================================================================
--- print-tree.c        (Revision 193925)
+++ print-tree.c        (Arbeitskopie)
@@ -255,7 +255,7 @@  print_node (FILE *file, const char *prefix, tree n
   /* Allow this function to be called if the table is not there.  */
   if (table)
     {
-      hash = ((unsigned long) node) % HASH_SIZE;
+      hash = ((uintptr_t) node) % HASH_SIZE;

       /* If node is in the table, just mention its address.  */
       for (b = table[hash]; b; b = b->next)