Re: [PR debug/53682] avoid crash in cselib promote_debug_loc

2012-06-25 Thread Jakub Jelinek
On Mon, Jun 25, 2012 at 07:32:04AM -0300, Alexandre Oliva wrote: > Unless there are objections in the next 24 hours, I'm going to install > this in the 4.7 branch too. Regstrapped on x86_64-linux-gnu and > i686-linux-gnu. Just install it right away. Thanks. > > for gcc/ChangeLog > > from Alex

Re: [PR debug/53682] avoid crash in cselib promote_debug_loc

2012-06-25 Thread Alexandre Oliva
On Jun 20, 2012, Alexandre Oliva wrote: > When promote_debug_loc was first introduced, it would never be called > with a NULL loc list. However, because of the strategy of temporarily > resetting loc lists before recursion introduced a few months ago in > alias.c, the earlier assumption no longe

Re: [PR debug/53682] avoid crash in cselib promote_debug_loc

2012-06-20 Thread Jakub Jelinek
On Wed, Jun 20, 2012 at 08:43:53PM -0300, Alexandre Oliva wrote: > On Jun 20, 2012, Jakub Jelinek wrote: > > > On Wed, Jun 20, 2012 at 12:39:29AM -0300, Alexandre Oliva wrote: > >> When promote_debug_loc was first introduced, it would never be called > >> with a NULL loc list. However, because o

Re: [PR debug/53682] avoid crash in cselib promote_debug_loc

2012-06-20 Thread Alexandre Oliva
On Jun 20, 2012, Jakub Jelinek wrote: > On Wed, Jun 20, 2012 at 12:39:29AM -0300, Alexandre Oliva wrote: >> When promote_debug_loc was first introduced, it would never be called >> with a NULL loc list. However, because of the strategy of temporarily >> resetting loc lists before recursion intro

Re: [PR debug/53682] avoid crash in cselib promote_debug_loc

2012-06-20 Thread Jakub Jelinek
On Wed, Jun 20, 2012 at 12:39:29AM -0300, Alexandre Oliva wrote: > When promote_debug_loc was first introduced, it would never be called > with a NULL loc list. However, because of the strategy of temporarily > resetting loc lists before recursion introduced a few months ago in > alias.c, the earl

[PR debug/53682] avoid crash in cselib promote_debug_loc

2012-06-19 Thread Alexandre Oliva
When promote_debug_loc was first introduced, it would never be called with a NULL loc list. However, because of the strategy of temporarily resetting loc lists before recursion introduced a few months ago in alias.c, the earlier assumption no longer holds. This patch adusts promote_debug_loc to d