[PATCH 1/3] power10: Add PCREL_OPT load support.

2020-09-04 Thread Michael Meissner via Gcc-patches
power10: Add PCREL_OPT load support. This patch adds support for optimizing power10 loads of an external variable to eliminate loading the address of the variable, and then doing a subsequent load using that address. The next patch will add the support for optimizing power10 stores to an external

Re: [PATCH 1/3] Power10: Add PCREL_OPT load support

2020-09-03 Thread Michael Meissner via Gcc-patches
On Thu, Aug 20, 2020 at 09:09:40PM -0500, Segher Boessenkool wrote: > Hi! > > On Tue, Aug 18, 2020 at 02:34:01AM -0400, Michael Meissner wrote: > > +// Maximum number of insns to scan between the load address and the load > > that > > Please don't mix /* and // comments. Just stick to /* commen

Re: [PATCH 1/3] Power10: Add PCREL_OPT load support

2020-08-20 Thread Segher Boessenkool
Hi! On Tue, Aug 18, 2020 at 02:34:01AM -0400, Michael Meissner wrote: > +// Maximum number of insns to scan between the load address and the load that Please don't mix /* and // comments. Just stick to /* comments, like all the rest of our backend? > +const int MAX_PCREL_OPT_INSNS= 10;

Re: [PATCH 1/3] Power10: Add PCREL_OPT load support

2020-08-17 Thread Michael Meissner via Gcc-patches
[PATCH 1/3] Power10: Add PCREL_OPT load support. This patch adds support for optimizing power10 loads of an external variable to eliminate loading the address of the variable, and then doing a subsequent load using that address. I have built compilers with and without these set of 3 patches