This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] PowerPC address support clean, patch 1 of 4
- From: Segher Boessenkool <segher at kernel dot crashing dot org>
- To: Michael Meissner <meissner at linux dot ibm dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>, David Edelsohn <dje dot gcc at gmail dot com>, Bill Schmidt <wschmidt at linux dot vnet dot ibm dot com>
- Date: Wed, 9 May 2018 17:54:53 -0500
- Subject: Re: [PATCH] PowerPC address support clean, patch 1 of 4
- References: <20180503171703.GA4233@ibm-toto.the-meissners.org>
Hi Mike,
On Thu, May 03, 2018 at 01:17:03PM -0400, Michael Meissner wrote:
> 2018-05-03 Michael Meissner <meissner@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