Re: [PR target/85401] initialize the move cost table before using it (in another place, too)

2019-10-04 Thread Jeff Law
On 10/4/19 7:23 AM, co...@sdf.org wrote: > On Tue, Oct 01, 2019 at 01:26:16PM -0600, Jeff Law wrote: >> On 9/30/19 2:45 PM, co...@sdf.org wrote: >>> On Mon, Sep 30, 2019 at 11:46:24AM -0400, Vladimir Makarov wrote: Yes, the patch is mostly ok.  You can commit it into the trunk after

[PR target/85401] initialize the move cost table before using it (in another place, too)

2019-10-04 Thread coypu
On Tue, Oct 01, 2019 at 01:26:16PM -0600, Jeff Law wrote: > On 9/30/19 2:45 PM, co...@sdf.org wrote: > > On Mon, Sep 30, 2019 at 11:46:24AM -0400, Vladimir Makarov wrote: > >> Yes, the patch is mostly ok.  You can commit it into the trunk after > >> applying changes mentioned below. If you do not

Re: [ping][PR target/85401] initialize the move cost table before using it

2019-10-01 Thread Jeff Law
On 9/30/19 2:45 PM, co...@sdf.org wrote: > On Mon, Sep 30, 2019 at 11:46:24AM -0400, Vladimir Makarov wrote: >> Yes, the patch is mostly ok.  You can commit it into the trunk after >> applying changes mentioned below. If you do not have a write access, let me >> know I'll commit the patch by

Re: [ping][PR target/85401] initialize the move cost table before using it

2019-09-30 Thread coypu
On Mon, Sep 30, 2019 at 11:46:24AM -0400, Vladimir Makarov wrote: > Yes, the patch is mostly ok.  You can commit it into the trunk after > applying changes mentioned below. If you do not have a write access, let me > know I'll commit the patch by myself. I don't have commit access. It would be

Re: [ping][PR target/85401] initialize the move cost table before using it

2019-09-30 Thread Vladimir Makarov
On 2019-09-28 5:52 a.m., co...@sdf.org wrote: re-posting, now CC'ing vmakarov who might be the right person to ping about issues in this file. apologies for the noise if I'm wrong. -- This seems to be the way the rest of ira-color.c does it. I hope it's OK. It does fix the segfault. Yes, the

[ping][PR target/85401] initialize the move cost table before using it

2019-09-28 Thread coypu
re-posting, now CC'ing vmakarov who might be the right person to ping about issues in this file. apologies for the noise if I'm wrong. -- This seems to be the way the rest of ira-color.c does it. I hope it's OK. It does fix the segfault. 2019-09-10 Maya Rashish PR target/85401

[PR target/85401] initialize the move cost table before using it

2019-09-14 Thread coypu
This seems to be the way the rest of ira-color.c does it. I hope it's OK. It does fix the segfault. 2019-09-10 Maya Rashish PR target/85401 * ira-color.c: (allocno_copy_cost_saving) Call ira_init_register_move_cost_if_necessary diff --git a/gcc/ira-color.c