Re: [PATCH, rs6000][GCC6] Fix PR78543, ICE in push_reload on powerpc64le-linux-gnu

2017-03-07 Thread Ulrich Weigand
Peter Bergner wrote: > If we look at rs6000_mode_dependent_address(), it accepts some addresses > as not being mode dependent: > > case PLUS: > /* Any offset from virtual_stack_vars_rtx and arg_pointer_rtx > is considered a legitimate address before reload, so there >

Re: [PATCH, rs6000][GCC6] Fix PR78543, ICE in push_reload on powerpc64le-linux-gnu

2017-03-06 Thread Segher Boessenkool
On Mon, Mar 06, 2017 at 12:16:58PM -0600, Peter Bergner wrote: [ big snip, thanks for the thorough explanation! ] > The following patch passes bootstrap and regtesting on powerpc64le-linux. > Ok for the GCC 6 branch? Please also test on BE and 32-bit. > We don't hit this on trunk, because we're

[PATCH, rs6000][GCC6] Fix PR78543, ICE in push_reload on powerpc64le-linux-gnu

2017-03-06 Thread Peter Bergner
PR78543 has two related test cases that have similar insns that need reloading (pseudo 185 in this case) due to spills: (insn 142 144 146 20 (parallel [ (set (reg:HI 4 4 [orig:260 p ] [260]) (bswap:HI (subreg/s/v:HI (reg:DI 185 [ load_dst_59 ]) 0))) (clobber