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 =

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

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

Re: [RFA] PowerPC e5500 and e6500 cores support

2012-06-06 Thread David Edelsohn
On Tue, Jun 5, 2012 at 3:13 PM, Edmar ed...@freescale.com 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 ed...@freescale.com    *

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 ed...@freescale.com * config/rs6000/rs6000-tables.opt: Regenerated. On 06/04/2012 08:45 PM,

Re: [RFA] PowerPC e5500 and e6500 cores support

2012-06-04 Thread David Edelsohn
On Fri, Jun 1, 2012 at 11:57 AM, Edmar ed...@freescale.com 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

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

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

Re: [RFA] PowerPC e5500 and e6500 cores support

2012-05-23 Thread David Edelsohn
On Wed, May 23, 2012 at 10:18 AM, Edmar ed...@freescale.com 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

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

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 boolean

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 always