Re: expand_expr tweaks to fix PR57134

2013-10-01 Thread Alan Modra
I'm committing this cleanup patch to my PR 57134,57586 changes as obvious. That it is obvious can be seen from an assert in tree-ssa-operands.c get_asm_expr_operands(). /* This should have been split in gimplify_asm_expr. */ gcc_assert (!allows_reg || !is_inout); Bootstrapped, etc.

Re: expand_expr tweaks to fix PR57134

2013-09-24 Thread Richard Biener
On Tue, Sep 24, 2013 at 12:43 PM, Alan Modra wrote: > On Fri, Sep 13, 2013 at 12:37:20PM +0930, Alan Modra wrote: >> PR middle-end/57586 >> * stmt.c (expand_asm_operands): Call expand_expr with >> EXPAND_MEMORY for output operands that disallow regs. Don't >> use EXPAND_WR

Re: expand_expr tweaks to fix PR57134

2013-09-24 Thread Alan Modra
On Fri, Sep 13, 2013 at 12:37:20PM +0930, Alan Modra wrote: > PR middle-end/57586 > * stmt.c (expand_asm_operands): Call expand_expr with > EXPAND_MEMORY for output operands that disallow regs. Don't > use EXPAND_WRITE on inout operands. Ping? -- Alan Modra Australia Dev

Re: expand_expr tweaks to fix PR57134

2013-09-12 Thread Alan Modra
This is a followup to http://gcc.gnu.org/ml/gcc-patches/2013-06/msg00837.html which is still lacking an OK. Apologies for dropping this patch on the floor. PR middle-end/57586 * stmt.c (expand_asm_operands): Call expand_expr with EXPAND_MEMORY for output operands that disa

Re: expand_expr tweaks to fix PR57134

2013-06-14 Thread Richard Biener
On Fri, Jun 14, 2013 at 10:38 AM, Alan Modra wrote: > On Thu, Jun 13, 2013 at 10:45:38AM +0200, Richard Biener wrote: >> On Wed, Jun 12, 2013 at 4:48 AM, Alan Modra wrote: >> > The following patch fixes PR57134 by >> > a) excluding bitfield expansion when EXPAND_MEMORY, and >> > b) passing down t

Re: expand_expr tweaks to fix PR57134

2013-06-14 Thread Alan Modra
On Thu, Jun 13, 2013 at 10:45:38AM +0200, Richard Biener wrote: > On Wed, Jun 12, 2013 at 4:48 AM, Alan Modra wrote: > > The following patch fixes PR57134 by > > a) excluding bitfield expansion when EXPAND_MEMORY, and > > b) passing down the EXPAND_MEMORY modifier in a couple of places where > > t

Re: expand_expr tweaks to fix PR57134

2013-06-13 Thread Richard Biener
On Wed, Jun 12, 2013 at 4:48 AM, Alan Modra wrote: > The following patch fixes PR57134 by > a) excluding bitfield expansion when EXPAND_MEMORY, and > b) passing down the EXPAND_MEMORY modifier in a couple of places where > this does not currently happen on recursive calls to expand_expr(). > > (a)

expand_expr tweaks to fix PR57134

2013-06-11 Thread Alan Modra
The following patch fixes PR57134 by a) excluding bitfield expansion when EXPAND_MEMORY, and b) passing down the EXPAND_MEMORY modifier in a couple of places where this does not currently happen on recursive calls to expand_expr(). (a) has precedent elsewhere in expr.c, eg. see expand_expr_real_1