Re: [Patch, PR 54128] ira.c change to fix mips bootstrap

2012-12-21 Thread Steve Ellcey
On Fri, 2012-12-21 at 08:46 +0100, Jakub Jelinek wrote: > Given Alex' comments in the PR, the second hunk is definitely ok for trunk, > the first one can be applied too (but you can skip it too if you want, it > shouldn't make a difference). Thanks, I checked in both chunks. Steve Ellcey sell..

Re: [Patch, PR 54128] ira.c change to fix mips bootstrap

2012-12-20 Thread Jakub Jelinek
On Fri, Aug 31, 2012 at 10:58:51AM -0700, Steve Ellcey wrote: > Here is my patch to fix the bootstrap comparision failure (PR 54128) on > MIPS. The reason for the comparision failure was a difference in > register usage and I tracked it down to build_insn_chain which checked > all instructions fo

Re: [Patch, PR 54128] ira.c change to fix mips bootstrap

2012-09-08 Thread Richard Sandiford
Don't have time to look at this in detail, but FWIW: Steve Ellcey writes: > On Wed, 2012-09-05 at 08:15 +0200, Jakub Jelinek wrote: >> On Fri, Aug 31, 2012 at 10:58:51AM -0700, Steve Ellcey wrote: >> > Here is my patch to fix the bootstrap comparision failure (PR 54128) on >> > MIPS. The reason

Re: [Patch, PR 54128] ira.c change to fix mips bootstrap

2012-09-07 Thread Steve Ellcey
On Wed, 2012-09-05 at 08:15 +0200, Jakub Jelinek wrote: > On Fri, Aug 31, 2012 at 10:58:51AM -0700, Steve Ellcey wrote: > > Here is my patch to fix the bootstrap comparision failure (PR 54128) on > > MIPS. The reason for the comparision failure was a difference in > > register usage and I tracked

Re: [Patch, PR 54128] ira.c change to fix mips bootstrap

2012-09-05 Thread Steve Ellcey
On Wed, 2012-09-05 at 08:15 +0200, Jakub Jelinek wrote: > The debug insns generally shouldn't extend the lifetime of pseudos (see the > valtrack.c stuff), so if you hit this, there is probably some earlier bug > that didn't reset/adjust the debug insns in question. > I'm not saying the ira.c patch

Re: [Patch, PR 54128] ira.c change to fix mips bootstrap

2012-09-04 Thread Jakub Jelinek
On Fri, Aug 31, 2012 at 10:58:51AM -0700, Steve Ellcey wrote: > Here is my patch to fix the bootstrap comparision failure (PR 54128) on > MIPS. The reason for the comparision failure was a difference in > register usage and I tracked it down to build_insn_chain which checked > all instructions fo

[Patch, PR 54128] ira.c change to fix mips bootstrap

2012-08-31 Thread Steve Ellcey
Here is my patch to fix the bootstrap comparision failure (PR 54128) on MIPS. The reason for the comparision failure was a difference in register usage and I tracked it down to build_insn_chain which checked all instructions for register usage in order to set the dead_or_set and live_relevant_regs