diff mbox

patch to fix PR70703

Message ID 8b578f82-f008-aacc-f3e6-c6bc248babb3@redhat.com
State New
Headers show

Commit Message

Vladimir Makarov April 5, 2017, 4:18 p.m. UTC
On 04/05/2017 12:07 PM, Vladimir Makarov wrote:
>
>
> I'll correct the patch.
>
Here is the patch I've committed.

2017-04-05  Vladimir Makarov  <vmakarov@redhat.com>

         PR rtl-optimization/70703
         * ira-color.c (update_conflict_hard_regno_costs): Use
         HOST_WIDE_INT instead of long.

                 cont_p = true;

Comments

Richard Biener April 6, 2017, 10:24 a.m. UTC | #1
On Wed, Apr 5, 2017 at 6:18 PM, Vladimir Makarov <vmakarov@redhat.com> wrote:
>
>
> On 04/05/2017 12:07 PM, Vladimir Makarov wrote:
>>
>>
>>
>> I'll correct the patch.
>>
> Here is the patch I've committed.

Note that in such contexts it's better to just use [u]int64_t.

Richard.

> 2017-04-05  Vladimir Makarov  <vmakarov@redhat.com>
>
>         PR rtl-optimization/70703
>         * ira-color.c (update_conflict_hard_regno_costs): Use
>         HOST_WIDE_INT instead of long.
>
> Index: ira-color.c
> ===================================================================
> --- ira-color.c (revision 246707)
> +++ ira-color.c (working copy)
> @@ -1522,7 +1522,7 @@
>                 index = ira_class_hard_reg_index[aclass][hard_regno];
>                 if (index < 0)
>                   continue;
> -               cost = (int) (((long) conflict_costs [i] * mult) / div);
> +               cost = (int) (((HOST_WIDE_INT) conflict_costs [i] * mult) /
> div);
>                 if (cost == 0)
>                   continue;
>                 cont_p = true;
>
diff mbox

Patch

Index: ira-color.c
===================================================================
--- ira-color.c (revision 246707)
+++ ira-color.c (working copy)
@@ -1522,7 +1522,7 @@ 
                 index = ira_class_hard_reg_index[aclass][hard_regno];
                 if (index < 0)
                   continue;
-               cost = (int) (((long) conflict_costs [i] * mult) / div);
+               cost = (int) (((HOST_WIDE_INT) conflict_costs [i] * 
mult) / div);
                 if (cost == 0)
                   continue;