[PATCH] Disable PowerPC pc-relative support until the code is checked in

Segher Boessenkool segher@kernel.crashing.org
Thu Jun 6 23:14:00 GMT 2019


Hi Mike,

On Thu, Jun 06, 2019 at 06:42:16PM -0400, Michael Meissner wrote:
> 2019-06-06  Michael Meissner  <meissner@linux.ibm.com>
> 
> 	* config/rs6000/rs6000-cpus.def (ISA_FUTURE_MASKS_SERVER): Delete
> 	enabling -mprefixed-addr and -mpcrel by default.

Why disable prefixed-addr?

> 	* config/rs6000/rs6000.c (rs6000_option_override_internal): Make
> 	-mpcrel and -mprefixed-addr act like other swtiches (i.e. using

Typo ("switches").

> 	-mpcrel automatically sets -mcpu=future and -mprefixed-addr, and

Automatically setting -mcpu= is a bad thing.  Instead, we should just
error out if someone tries to use -mpcrel with a CPU (or ABI, etc.) that
doesn't support it.  Or, is there any special reason you want it?

In the future, we will not have an -mprefixed-addr option (it will be
always on for CPUs that support it), and I don't see any real reason
to allow disabling pcrel either, but we'll see.


Segher



More information about the Gcc-patches mailing list