Re: RFA[powerpc]: patch to fix PR79916

2018-04-13 Thread Vladimir Makarov
On 04/13/2018 05:26 PM, Segher Boessenkool wrote: Hi! On Fri, Apr 13, 2018 at 04:43:02PM -0400, Vladimir Makarov wrote: On 04/13/2018 03:58 PM, Alexander Monakov wrote: Here's another compact variant: regno = reg_renumber[regno]; if (regno < 0) regno = cl ==

Re: RFA[powerpc]: patch to fix PR79916

2018-04-13 Thread Segher Boessenkool
Hi! On Fri, Apr 13, 2018 at 04:43:02PM -0400, Vladimir Makarov wrote: > On 04/13/2018 03:58 PM, Alexander Monakov wrote: > >Here's another compact variant: > > > > regno = reg_renumber[regno]; > > if (regno < 0) > > regno = cl == NO_REGS ? -1 : ira_class_hard_regs[cl][1]; > > >

Re: RFA[powerpc]: patch to fix PR79916

2018-04-13 Thread Vladimir Makarov
On 04/13/2018 03:58 PM, Alexander Monakov wrote: On Fri, 13 Apr 2018, Jakub Jelinek wrote: if (reg_renumber[regno] >= 0) regno = reg_renumber[regno]; else regno = cl == NO_REGS ? -1 : ira_class_hard_regs[cl][1]; or regno = (reg_renumber[reg

Re: RFA[powerpc]: patch to fix PR79916

2018-04-13 Thread Alexander Monakov
On Fri, 13 Apr 2018, Jakub Jelinek wrote: > if (reg_renumber[regno] >= 0) > regno = reg_renumber[regno]; > else > regno = cl == NO_REGS ? -1 : ira_class_hard_regs[cl][1]; > or > regno = (reg_renumber[regno] >= 0 > ? reg_renumber[regno] >

Re: RFA[powerpc]: patch to fix PR79916

2018-04-13 Thread Jakub Jelinek
On Fri, Apr 13, 2018 at 03:29:47PM -0400, Vladimir Makarov wrote: >   The attached patch fixes > > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79916 > >   The PR is about LRA cycling on some tests when SD data are used.  The > problem was in that actual assigned reg to pseudo was not in the pseu

RFA[powerpc]: patch to fix PR79916

2018-04-13 Thread Vladimir Makarov
  The attached patch fixes https://gcc.gnu.org/bugzilla/show_bug.cgi?id=79916   The PR is about LRA cycling on some tests when SD data are used.  The problem was in that actual assigned reg to pseudo was not in the pseudo preferred class and this resulted in wrong generated code which LRA tri

Re: [RFA] PowerPC e5500 and e6500 cores support

2012-06-06 Thread David Edelsohn
On Tue, Jun 5, 2012 at 3:13 PM, Edmar wrote: > The patch I submitted had an omission. I failed to regenerate > rs6000-tables.opt > (Sorry, I misunderstood gcc_update --touch instructions) > OK to commit the update ? > > 2012-06-05  Edmar Wienskoski > >    * config/rs6000/rs6000-tables.opt: Regene

Re: [RFA] PowerPC e5500 and e6500 cores support

2012-06-05 Thread Edmar
The patch I submitted had an omission. I failed to regenerate rs6000-tables.opt (Sorry, I misunderstood gcc_update --touch instructions) OK to commit the update ? 2012-06-05 Edmar Wienskoski * config/rs6000/rs6000-tables.opt: Regenerated. On 06/04/2012 08:45 PM, David Edelsohn wrote: T

Re: [RFA] PowerPC e5500 and e6500 cores support

2012-06-04 Thread David Edelsohn
On Fri, Jun 1, 2012 at 11:57 AM, Edmar wrote: > Freescale would like to contribute these patches to gcc. > > It enables gcc for the new Freescale 64 bit cores. It creates a pipeline > description, and set proper default flags for the e5500 and e6500 cores. > > Some Altivec extensions for e6500 wil

Re: [RFA] PowerPC e5500 and e6500 cores support

2012-06-01 Thread Edmar
Freescale would like to contribute these patches to gcc. It enables gcc for the new Freescale 64 bit cores. It creates a pipeline description, and set proper default flags for the e5500 and e6500 cores. Some Altivec extensions for e6500 will be submitted as a separate process. The patch was reg

Re: [RFA] PowerPC e5500 and e6500 cores support

2012-05-23 Thread David Edelsohn
On Wed, May 23, 2012 at 10:18 AM, Edmar wrote: > David, Michael, > > Thanks for the feedback. > If you don't object, I will relay the message to the designers. > > Meanwhile I have to work with the cards I have, so... > I will break the patch in three parts: > - One that includes the very basic, s

Re: [RFA] PowerPC e5500 and e6500 cores support

2012-05-23 Thread Edmar
David, Michael, Thanks for the feedback. If you don't object, I will relay the message to the designers. Meanwhile I have to work with the cards I have, so... I will break the patch in three parts: - One that includes the very basic, scheduling etc. - One for the Altivec builtins, which I will h

Re: [RFA] PowerPC e5500 and e6500 cores support

2012-05-21 Thread David Edelsohn
> Regarding the implementation of popcntb, popcntd, and cmpb. Gcc has > dedicated masks on target_flags for them, but due to shortage of bits, > those masks controls more than the name implies. The target flag bits control more than the name implies, but the bits correspond to published ISA levels

Re: [RFA] PowerPC e5500 and e6500 cores support

2012-05-18 Thread Edmar
Michael, Thanks for reviewing the patch and all the suggestions. I have some questions / comments bellow. Regards, Edmar On 05/17/2012 06:16 PM, Michael Meissner wrote: In the patch I minimized the number of changes, while not adding any new mask to target_flags. While we may get some bits

Re: [RFA] PowerPC e5500 and e6500 cores support

2012-05-17 Thread Michael Meissner
On Tue, Mar 06, 2012 at 11:44:08AM -0500, Edmar wrote: > Freescale would like to contribute these patches to gcc. > > It enables gcc for the new Freescale 64 bit cores. It creates a pipeline > description, and set proper default flags for the e5500 and e6500 cores. > > Both are 64 bit cores capa

Re: PING [RFA] PowerPC e5500 and e6500 cores support

2012-05-16 Thread Edmar
Joseph, Thanks for the tip. It is likely I need to re-generate the patch (and re-test) as there had been a lot of activity in the trunk since the original post. I will do this soon. Regards, Edmar On 05/16/2012 11:10 AM, Joseph S. Myers wrote: On Fri, 16 Mar 2012, Edmar wrote: I am pingin

Re: PING [RFA] PowerPC e5500 and e6500 cores support

2012-05-16 Thread Joseph S. Myers
On Fri, 16 Mar 2012, Edmar wrote: > I am pinging this post: > > http://gcc.gnu.org/ml/gcc-patches/2012-03/msg00430.html I don't see any sign of anything further having happened with this patch ... you should ping approximately weekly until a patch is reviewed, and probably also CC relevant mai

PING [RFA] PowerPC e5500 and e6500 cores support

2012-03-16 Thread Edmar
I am pinging this post: http://gcc.gnu.org/ml/gcc-patches/2012-03/msg00430.html Thanks Edmar

Re: [RFA] [PowerPC]

2011-04-21 Thread edmar
On 04/20/2011 07:52 PM, Segher Boessenkool wrote: The test and-1.c has wrong logic. In the formula: y & ~(y & -y) The part (y & -y) is always a mask with one bit set, which corresponds to the least significant "1" bit in y. The final result is that bit, is set to zero (y & ~mask) There is no bo

Re: [RFA] [PowerPC]

2011-04-20 Thread Segher Boessenkool
The test and-1.c has wrong logic. In the formula: y & ~(y & -y) The part (y & -y) is always a mask with one bit set, which corresponds to the least significant "1" bit in y. The final result is that bit, is set to zero (y & ~mask) There is no boolean simplification possible, and the compiler alw

[RFA] [PowerPC]

2011-04-19 Thread edmar
This patch fixes some test cases for PowerPC. The tests pr39902-2.c, dfp-dd.c, and dfp-td.c reports as errors when gcc is configured without dfp support. This patch will make the tests to be reported as unsupported. The test and-1.c has wrong logic. In the formula: y & ~(y & -y) The part (y & -