Re: [Patch, ira] Invalid assert in reload1.c::finish_spills?

2019-08-12 Thread Vladimir Makarov
On 2019-08-09 5:23 p.m., Jeff Law wrote: I've committed it after letting it bootstrap/regression test on ppc64le, aarch64 and others. It also didn't regress on any of the *-elf targets I'm testing which are far more likely to exercise this code. Thank you, Jeff.

Re: [Patch, ira] Invalid assert in reload1.c::finish_spills?

2019-08-09 Thread Jeff Law
On 8/7/19 8:15 AM, Vladimir Makarov wrote: > On 8/7/19 7:36 AM, senthilkumar.selva...@microchip.com wrote: >> Hi, >> >>    gcc/testsuite/c-c++-common/pr60101.c fails with an ICE for the >>    avr target, because of a gcc_assert firing at reload1.c:4233 >> >>    The assert (in the patch below)

Re: [Patch, ira] Invalid assert in reload1.c::finish_spills?

2019-08-08 Thread Jeff Law
On 8/7/19 8:15 AM, Vladimir Makarov wrote: > On 8/7/19 7:36 AM, senthilkumar.selva...@microchip.com wrote: >> Hi, >> >>    gcc/testsuite/c-c++-common/pr60101.c fails with an ICE for the >>    avr target, because of a gcc_assert firing at reload1.c:4233 >> >>    The assert (in the patch below)

Re: [Patch, ira] Invalid assert in reload1.c::finish_spills?

2019-08-07 Thread Vladimir Makarov
On 8/7/19 7:36 AM, senthilkumar.selva...@microchip.com wrote: Hi, gcc/testsuite/c-c++-common/pr60101.c fails with an ICE for the avr target, because of a gcc_assert firing at reload1.c:4233 The assert (in the patch below) looks bogus to me, as it's in the if block of if (!

Re: [Patch, ira] Invalid assert in reload1.c::finish_spills?

2019-08-07 Thread Michael Matz
Hi, On Wed, 7 Aug 2019, senthilkumar.selva...@microchip.com wrote: > gcc/testsuite/c-c++-common/pr60101.c fails with an ICE for the > avr target, because of a gcc_assert firing at reload1.c:4233 > > The assert (in the patch below) looks bogus to me, as it's in > the if block of > >

[Patch, ira] Invalid assert in reload1.c::finish_spills?

2019-08-07 Thread SenthilKumar.Selvaraj
Hi, gcc/testsuite/c-c++-common/pr60101.c fails with an ICE for the avr target, because of a gcc_assert firing at reload1.c:4233 The assert (in the patch below) looks bogus to me, as it's in the if block of if (! ira_conflicts_p || reg_renumber[i] >= 0) For this testcase and for