Re: [PATCH], Set -mpcrel by default on Linux 64-bit systems for -mcpu=future

2020-04-02 Thread Segher Boessenkool
On Fri, Mar 27, 2020 at 02:30:36PM -0400, Michael Meissner wrote: > > > -/* Support for a future processor's features. Do not enable -mpcrel > > > until it > > > - is fully functional. */ > > > +/* Support for a future processor's features. The addressing related > > > options > > > + (lik

Re: [PATCH], Set -mpcrel by default on Linux 64-bit systems for -mcpu=future

2020-03-27 Thread Michael Meissner via Gcc-patches
On Wed, Mar 25, 2020 at 11:07:04AM -0500, will schmidt wrote: > Hi, > Comments inline. > > > Is this patch acceptable to be committed to the master branch. I have done > > various tests with this patch, including most recently bootstraping and > > running > > make check. I have built the Spec 2

Re: [PATCH], Set -mpcrel by default on Linux 64-bit systems for -mcpu=future

2020-03-25 Thread Segher Boessenkool
On Wed, Mar 25, 2020 at 11:07:04AM -0500, will schmidt wrote: > Subject: [Patch v327] set -mcprel by default ... > > Include some powerpc or rs6000 reference in the subject. My filters > missed this one. (I'm assuming this is powerpc target specific). rs6000: Enable pcrel by default (etc.) S

Re: [PATCH], Set -mpcrel by default on Linux 64-bit systems for -mcpu=future

2020-03-25 Thread will schmidt via Gcc-patches
Hi, Comments inline. (Taking a pass with focus on cosmetic stuff. This is intended to help Segher focus on the harder parts :-) ). On Mon, 2020-03-23 at 16:38 -0400, Michael Meissner via Gcc-patches wrote: Subject: [Patch v327] set -mcprel by default ... Include some powerpc or rs6000 referenc

[PATCH], Set -mpcrel by default on Linux 64-bit systems for -mcpu=future

2020-03-23 Thread Michael Meissner via Gcc-patches
This is a revision of a patch that I've submitted several times. It makes -mpcrel the default on Linux 64-bit systems that use ELF v2, use the medium code mode, and if the user did not disable prefixed load/store instructions for -mcpu=future. Previous versions of the patch had two macros that th