RE: [PATCH][MIPS] Fix P5600 memory cost

2014-12-09 Thread Prachi Godbole
t: Re: [PATCH][MIPS] Fix P5600 memory cost On 12/09/14 15:33, Matthew Fortune wrote: > Hi Prachi, > > I'm afraid you updated the wrong Changelog with this commit. GCC > changes are recorded in gcc/ChangeLog. > > I'm not sure what the correct procedure is for fixing this

Re: [PATCH][MIPS] Fix P5600 memory cost

2014-12-09 Thread Jeff Law
On 12/09/14 15:33, Matthew Fortune wrote: Hi Prachi, I'm afraid you updated the wrong Changelog with this commit. GCC changes are recorded in gcc/ChangeLog. I'm not sure what the correct procedure is for fixing this. Jeff: Should a mistake like this be fixed by removing the entry from the top

RE: [PATCH][MIPS] Fix P5600 memory cost

2014-12-09 Thread Matthew Fortune
ppropriate location in the gcc/ChangeLog one? Thanks, Matthew > -Original Message- > From: Prachi Godbole > Sent: 26 November 2014 08:40 > To: Matthew Fortune; gcc-patches@gcc.gnu.org > Subject: RE: [PATCH][MIPS] Fix P5600 memory cost > > Committed with Cha

RE: [PATCH][MIPS] Fix P5600 memory cost

2014-11-26 Thread Prachi Godbole
Committed with ChangeLog entry fixes. Prachi -Original Message- From: Matthew Fortune Sent: Wednesday, November 5, 2014 4:07 PM To: Prachi Godbole; gcc-patches@gcc.gnu.org Subject: RE: [PATCH][MIPS] Fix P5600 memory cost > The patch below fixes the memory cost for P5

RE: [PATCH][MIPS] Fix P5600 memory cost

2014-11-25 Thread Prachi Godbole
PM To: Prachi Godbole; gcc-patches@gcc.gnu.org Subject: RE: [PATCH][MIPS] Fix P5600 memory cost Hi Prachi, > OK with fixes to the changelog entry: > > latency not latency. Remember to tab in the changelog entry and split > the line as it will exceed 80 chars. Also two spaces between the

RE: [PATCH][MIPS] Fix P5600 memory cost

2014-11-25 Thread Matthew Fortune
Hi Prachi, > OK with fixes to the changelog entry: > > latency not latency. Remember to tab in the changelog entry and split the > line as it will exceed 80 chars. Also two spaces between the date/name and > name/email. E.g. > > 2014-11-05 Prachi Godbole > > * config/mips/mips.c (mips

RE: [PATCH][MIPS] Fix P5600 memory cost

2014-11-05 Thread Matthew Fortune
> The patch below fixes the memory cost for P5600. > > ChangeLog: > 2014-11-05 Prachi Godbole > > * config/mips/mips.c (mips_rtx_cost_data): Fix memory_letency cost for > p5600. Please follow these instructions to add yourself to MAINTAINERS in the write-after-approval section now that you have

[PATCH][MIPS] Fix P5600 memory cost

2014-11-04 Thread Prachi Godbole
Hi, The patch below fixes the memory cost for P5600. ChangeLog: 2014-11-05 Prachi Godbole * config/mips/mips.c (mips_rtx_cost_data): Fix memory_letency cost for p5600. diff --git a/gcc/config/mips/mips.c b/gcc/config/mips/mips.c index af6a913..558ba2f 100644 --- a/gcc/config/mips/mips.c +++