Hi Mike, On Thu, May 03, 2018 at 01:17:03PM -0400, Michael Meissner wrote: > 2018-05-03 Michael Meissner <meiss...@linux.vnet.ibm.com> > > * config/rs6000/rs6000.c (mode_supports_dq_form): Rename > mode_supports_vsx_dform_quad to mode_supports_dq_form. > (mode_supports_vsx_dform_quad): Likewise. > (quad_address_p): Likewise. > (reg_offset_addressing_ok_p): Likewise. > (offsettable_ok_by_alignment): Likewise. > (rs6000_legitimate_offset_address_p): Likewise. > (legitimate_lo_sum_address_p): Likewise. > (rs6000_legitimize_address): Likewise. > (rs6000_legitimize_reload_address): Likewise. > (rs6000_secondary_reload_inner): Likewise. > (rs6000_preferred_reload_class): Likewise. > (rs6000_output_move_128bit): Likewise.
* config/rs6000/rs6000.c (mode_supports_vsx_dform_quad): Rename to ... (mode_supports_dq_form): ... this. Update all callers. > --- gcc/config/rs6000/rs6000.c (revision 259864) > +++ gcc/config/rs6000/rs6000.c (working copy) > @@ -649,7 +649,7 @@ mode_supports_vmx_dform (machine_mode mo > is more limited than normal d-form addressing in that the offset must be > aligned on a 16-byte boundary. */ > static inline bool > -mode_supports_vsx_dform_quad (machine_mode mode) > +mode_supports_dq_form (machine_mode mode) > { > return ((reg_addr[mode].addr_mask[RELOAD_REG_ANY] & RELOAD_REG_QUAD_OFFSET) > != 0); Will this eventually handle all DQ-form, not just vector? Is it supposed to? Okay for trunk with the changelog fixed. Thanks! Segher