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 P5600.
> 
> ChangeLog:
> 2014-11-05 Prachi Godbole <prachi.godb...@imgtec.com>
> 
> * 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 write access to GCC:

https://gcc.gnu.org/svnwrite.html#authenticated

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  <prachi.godb...@imgtec.com>

        * config/mips/mips.c (mips_rtx_cost_data): Fix memory_latency cost for
        p5600.

Thanks,
Matthew

> diff --git a/gcc/config/mips/mips.c b/gcc/config/mips/mips.c index 
> af6a913..558ba2f 100644
> --- a/gcc/config/mips/mips.c
> +++ b/gcc/config/mips/mips.c
> @@ -1193,7 +1193,7 @@ static const struct mips_rtx_cost_data
>      COSTS_N_INSNS (8),            /* int_div_si */
>      COSTS_N_INSNS (8),            /* int_div_di */
>                     2,            /* branch_cost */
> -                  10             /* memory_latency */
> +                   4             /* memory_latency */
>    }
>  };
>  ^L

Reply via email to