Hi Mike,

On Thu, Jun 06, 2019 at 06:42:16PM -0400, Michael Meissner wrote:
> 2019-06-06  Michael Meissner  <meiss...@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

Reply via email to