On Tue, Apr 21, 2020 at 02:03:34PM -0400, Michael Meissner wrote:
> On Tue, Apr 07, 2020 at 05:58:01PM -0500, Segher Boessenkool wrote:
> > > + /* Enable -mprefixed by default on 64-bit 'future' systems. */
> > > + if (TARGET_FUTURE && TARGET_POWERPC64
> > > + && (rs6000_isa_flags_explicit
On Tue, Apr 07, 2020 at 05:58:01PM -0500, Segher Boessenkool wrote:
> > + /* Enable -mprefixed by default on 64-bit 'future' systems. */
> > + if (TARGET_FUTURE && TARGET_POWERPC64
> > + && (rs6000_isa_flags_explicit & OPTION_MASK_PREFIXED) == 0)
> > +rs6000_isa_flags |= OPTION_MASK_PRE
Hi!
On Mon, Apr 06, 2020 at 12:52:04PM -0400, Michael Meissner wrote:
> Commit message:
> Enable -mpcrel for -mcpu=future if it is allowed by the ABI.
You can just put this in the mail subject. No dot at the end, but start
with a capital, an put a subsystem label on it:
Subject: rs6000: Enable
On Mon, 2020-04-06 at 12:52 -0400, Michael Meissner via Gcc-patches wrote:
Hi,
Just a single extra blank line below.
I'm still not a fan of the "Do not set..." comment, but will assume there
is history that necessitates the comment.
Other sections looked OK to me.
Over to Segher. :-)
Thanks,
Commit message:
Enable -mpcrel for -mcpu=future if it is allowed by the ABI.
2020-04-06 Michael Meissner
* config/rs6000/linux64.h (PCREL_SUPPORTED_BY_OS): Enable
prefixed PC-relative addressing if the ABI supports it.
* config/rs6000/rs6000-cpus.def (ISA_FUTURE_MASKS_S