Re: [PATCH, V4] PowerPC Turn on -mpcrel by default for -mcpu=future

2020-04-21 Thread Segher Boessenkool
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

Re: [PATCH, V4] PowerPC Turn on -mpcrel by default for -mcpu=future

2020-04-21 Thread Michael Meissner via Gcc-patches
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

Re: [PATCH, V4] PowerPC Turn on -mpcrel by default for -mcpu=future

2020-04-07 Thread Segher Boessenkool
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

Re: [PATCH, V4] PowerPC Turn on -mpcrel by default for -mcpu=future

2020-04-06 Thread will schmidt via Gcc-patches
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,

[PATCH, V4] PowerPC Turn on -mpcrel by default for -mcpu=future

2020-04-06 Thread Michael Meissner via Gcc-patches
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