Re: [PATCH], Remove power9 fusion support, version 2

2018-11-16 Thread Segher Boessenkool
Hi Mike, Thanks for the changes. On Thu, Nov 08, 2018 at 04:28:52PM -0500, Michael Meissner wrote: > * config/rs6000/constraints.md (wF constraint): Update constraint > documentation for power8 fusion only. "so it is for ...", maybe? It is hard to understand your sentence. > +/

Re: [PATCH], Remove power9 fusion support

2018-11-14 Thread Segher Boessenkool
On Wed, Nov 07, 2018 at 01:36:50PM -0500, Michael Meissner wrote: > On Mon, Nov 05, 2018 at 04:09:23PM -0600, Segher Boessenkool wrote: > > Hi Mike, > > > > On Fri, Nov 02, 2018 at 02:37:34PM -0400, Michael Meissner wrote: > > > This patch removes all of the so-called power9 fusion support for the

[PATCH], Remove power9 fusion support, version 2

2018-11-08 Thread Michael Meissner
This is version 2 of the patch to remove power9 fusion. Is it ok to check into the trunk? [gcc] 2018-11-08 Michael Meissner * config/rs6000/constraints.md (wF constraint): Update constraint documentation for power8 fusion only. * config/rs6000/predicates.md (p9_fusion

Re: [PATCH], Remove power9 fusion support

2018-11-07 Thread Michael Meissner
On Mon, Nov 05, 2018 at 04:09:23PM -0600, Segher Boessenkool wrote: > Hi Mike, > > On Fri, Nov 02, 2018 at 02:37:34PM -0400, Michael Meissner wrote: > > This patch removes all of the so-called power9 fusion support for the GCC > > compiler. It leaves -mpower9-fusion as a deprecated switch in case

Re: [PATCH], Remove power9 fusion support

2018-11-05 Thread Segher Boessenkool
Hi Mike, On Fri, Nov 02, 2018 at 02:37:34PM -0400, Michael Meissner wrote: > This patch removes all of the so-called power9 fusion support for the GCC > compiler. It leaves -mpower9-fusion as a deprecated switch in case somebody > used it (the switch was never documented). As Mike Stump says, pl

Re: [PATCH], Remove power9 fusion support

2018-11-05 Thread Mike Stump
On Nov 2, 2018, at 11:37 AM, Michael Meissner wrote: > > As I discussed in my 2018 Cauldron talk, the PowerPC GCC compiler supported a > subset of the original design for fusion in the power9 hardware using > peepholes > to fuse together ADDIS instructions and floating point load/store operation

[PATCH], Remove power9 fusion support

2018-11-02 Thread Michael Meissner
As I discussed in my 2018 Cauldron talk, the PowerPC GCC compiler supported a subset of the original design for fusion in the power9 hardware using peepholes to fuse together ADDIS instructions and floating point load/store operations. However, while fusion was part of the original power9 design,