This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] PowerPC cleanup, remove -mpower9-dform{,-scalar,-vector} options
- From: Segher Boessenkool <segher at kernel dot crashing dot org>
- To: Michael Meissner <meissner at linux dot vnet 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: Thu, 24 Aug 2017 04:38:28 -0500
- Subject: Re: [PATCH] PowerPC cleanup, remove -mpower9-dform{,-scalar,-vector} options
- Authentication-results: sourceware.org; auth=none
- References: <20170823173443.GA14261@ibm-tiger.the-meissners.org>
Hi Mike,
On Wed, Aug 23, 2017 at 01:34:43PM -0400, Michael Meissner wrote:
> This patch eliminates the undocumented debugging options -mpower9-dform,
> -mpower9-dform-scalar, and -mpower9-dform-vector. These switches were added
> when I added the support for the ISA 3.0 (power9) d-form (register+offset)
> vector addressing.
> * config/rs6000/rs6000.c (rs6000_setup_reg_addr_masks): Change
> tests against -mpowwer9-dform* to -mpower9-vector. Delete code
Typo (powwwwwer!)
> @@ -4239,8 +4239,7 @@ rs6000_option_override_internal (bool gl
>
> /* For the newer switches (vsx, dfp, etc.) set some of the older options,
> unless the user explicitly used the -mno-<option> to disable the code. */
> - if (TARGET_P9_VECTOR || TARGET_MODULO || TARGET_P9_DFORM_SCALAR
> - || TARGET_P9_DFORM_VECTOR || TARGET_P9_DFORM_BOTH > 0)
> + if (TARGET_P9_VECTOR || TARGET_MODULO || TARGET_P9_MISC)
> rs6000_isa_flags |= (ISA_3_0_MASKS_SERVER & ~ignore_masks);
> else if (TARGET_P9_MINMAX)
> {
Where did TARGET_P9_MISC come from? It should be mentioned in the
changelog, if it is needed here.
Okay for trunk with those things dealt with. Thanks!
Segher