Re: [PATCH] Fix split_live_ranges_for_shrink_wrap (PR rtl-optimization/59166)

2013-11-27 Thread Martin Jambor
On Tue, Nov 26, 2013 at 09:25:40PM +0100, Jakub Jelinek wrote: > Hi! > > The problem on this testcase is that we have > (debug_insn 30 29 31 7 (var_location:HI D#1 (subreg:HI (reg/v:SI 93 [ p ]) > 0)) pr59166.c:20 -1 > (nil)) > and split_live_ranges_for_shrink_wrap decides to replace > SImod

Re: [PATCH] Fix split_live_ranges_for_shrink_wrap (PR rtl-optimization/59166)

2013-11-26 Thread Jeff Law
On 11/26/13 13:25, Jakub Jelinek wrote: Hi! The problem on this testcase is that we have (debug_insn 30 29 31 7 (var_location:HI D#1 (subreg:HI (reg/v:SI 93 [ p ]) 0)) pr59166.c:20 -1 (nil)) and split_live_ranges_for_shrink_wrap decides to replace SImode pseudo 93 with some other SImode p

Re: [PATCH] Fix split_live_ranges_for_shrink_wrap (PR rtl-optimization/59166)

2013-11-26 Thread Jeff Law
On 11/26/13 13:34, Steven Bosscher wrote: On Tue, Nov 26, 2013 at 9:25 PM, Jakub Jelinek wrote: \> PR rtl-optimization/59166 * ira.c (find_moveable_pseudos): Use DF_REF_REAL_LOC instead of DF_REF_LOC in validate_change call. (split_live_ranges_for_shrink_wrap):

Re: [PATCH] Fix split_live_ranges_for_shrink_wrap (PR rtl-optimization/59166)

2013-11-26 Thread Steven Bosscher
On Tue, Nov 26, 2013 at 9:34 PM, Steven Bosscher wrote: > On Tue, Nov 26, 2013 at 9:25 PM, Jakub Jelinek wrote: > \> PR rtl-optimization/59166 >> * ira.c (find_moveable_pseudos): Use DF_REF_REAL_LOC instead of >> DF_REF_LOC in validate_change call. >> (split_live_ran

Re: [PATCH] Fix split_live_ranges_for_shrink_wrap (PR rtl-optimization/59166)

2013-11-26 Thread Steven Bosscher
On Tue, Nov 26, 2013 at 9:25 PM, Jakub Jelinek wrote: \> PR rtl-optimization/59166 > * ira.c (find_moveable_pseudos): Use DF_REF_REAL_LOC instead of > DF_REF_LOC in validate_change call. > (split_live_ranges_for_shrink_wrap): Likewise. > > * gcc.dg/torture/pr

[PATCH] Fix split_live_ranges_for_shrink_wrap (PR rtl-optimization/59166)

2013-11-26 Thread Jakub Jelinek
Hi! The problem on this testcase is that we have (debug_insn 30 29 31 7 (var_location:HI D#1 (subreg:HI (reg/v:SI 93 [ p ]) 0)) pr59166.c:20 -1 (nil)) and split_live_ranges_for_shrink_wrap decides to replace SImode pseudo 93 with some other SImode pseudo. But it uses DF_REF_LOC, which is ad