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,