On 07/13/2017 02:58 AM, Richard Sandiford wrote: > This patch rewrites the condition: > > if (tmode != address_mode && tmode != pointer_mode) > tmode = address_mode; > > to the equivalent: > > tmode == pointer_mode ? pointer_mode : address_mode > > The latter has the advantage that the result is naturally > a scalar_int_mode; a later mechanical patch makes it one. > > 2017-07-13 Richard Sandiford <richard.sandif...@linaro.org> > Alan Hayward <alan.hayw...@arm.com> > David Sherwood <david.sherw...@arm.com> > > gcc/ > * expr.c (expand_expr_addr_expr): Add a new_tmode local variable > that is always either address_mode or pointer_mode. OK. jeff