Re: [PATCH] RX TARGET_RTX_COSTS function

2018-02-22 Thread Oleg Endo
On Thu, 2018-02-22 at 15:41 +, Nick Clifton wrote: > > > gcc/ChangeLog: > > > * config/rx/rx.c (rx_rtx_costs): New function. > > > (TARGET_RTX_COSTS): Override to use rx_rtx_costs. > Approved - please apply. > Thanks.  Committed as r257905. Cheers, Oleg

RE: [PATCH] RX TARGET_RTX_COSTS function

2018-02-22 Thread Sebastian Perta
stian > -Original Message- > From: Nick Clifton [mailto:ni...@redhat.com] > Sent: 22 February 2018 15:41 > To: Oleg Endo ; gcc-patches@gcc.gnu.org > Cc: Sebastian Perta ; 'DJ Delorie' > > Subject: Re: [PATCH] RX TARGET_RTX_COSTS function > > Hi Oleg,

Re: [PATCH] RX TARGET_RTX_COSTS function

2018-02-22 Thread Nick Clifton
Hi Oleg, > Ping. Sorry - I am not very good at spotting RX bugs on the gcc-patches list. :-( >> gcc/ChangeLog: >> * config/rx/rx.c (rx_rtx_costs): New function. >> (TARGET_RTX_COSTS): Override to use rx_rtx_costs. Approved - please apply. Cheers Nick

Re: [PATCH] RX TARGET_RTX_COSTS function

2018-02-22 Thread Oleg Endo
Ping. On Thu, 2018-02-15 at 23:07 +0900, Oleg Endo wrote: > On Wed, 2018-02-14 at 01:06 +0900, Oleg Endo wrote: > > > >   > > Do you happen to have any other numbers on the resulting code > > size/speed?  Looking at the new costs that the patch introduces, > > I'd > > expect there to be some more

Re: [PATCH] RX TARGET_RTX_COSTS function

2018-02-15 Thread Oleg Endo
On Wed, 2018-02-14 at 01:06 +0900, Oleg Endo wrote: >  > Do you happen to have any other numbers on the resulting code > size/speed?  Looking at the new costs that the patch introduces, I'd > expect there to be some more changes than just the 1/x... > I've checked your proposed patch with the CSi

Re: [PATCH] RX TARGET_RTX_COSTS function

2018-02-13 Thread Oleg Endo
Hi, On Tue, 2018-02-13 at 15:54 +, Sebastian Perta wrote: >  > The patch required some changes (the prototype, second param more > exactly, has changed) in order to compile in the trunk. Could you please send the patch as an attachment?  The formatting looks a bit off (tabs spaces etc). > So

[PATCH] RX TARGET_RTX_COSTS function

2018-02-13 Thread Sebastian Perta
Hello, DJ has posted a patch a few years ago which implements TARGET_RTX_COSTS for RX: https://gcc.gnu.org/ml/gcc-patches/2012-05/msg8.html Nick has accepted the patch: https://gcc.gnu.org/ml/gcc-patches/2012-05/msg00012.html But it was never made it into the trunk. The patch required some