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