RE: [PATCH] [X86_64] Fix alignment for znver1 arch.

[Kumar, Venkataramanan]

Thanks Uros,

I committed on Amit's behalf.
https://gcc.gnu.org/viewcvs/gcc?view=revision=245423

regards,
venkat.

> -Original Message-
> From: Uros Bizjak [mailto:ubiz...@gmail.com]
> Sent: Tuesday, February 14, 2017 2:43 PM
> To: Pawar, Amit <amit.pa...@amd.com>
> Cc: gcc-patches@gcc.gnu.org; Kumar, Venkataramanan
> <venkataramanan.ku...@amd.com>
> Subject: Re: [PATCH] [X86_64] Fix alignment for znver1 arch.
> 
> On Tue, Feb 14, 2017 at 8:48 AM, Pawar, Amit <amit.pa...@amd.com> wrote:
> > Hi maintainers,
> >
> > Please find the below patch which changes the code alignment values for
> znver1. Bootstrap and regression test passed on x86_64.
> > OK to apply?
> 
> OK.
> 
> Thanks,
> Uros.
> 
> > Thanks,
> > Amit Pawar
> >
> > 
> > diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 2561d53..a5b0159
> > 100644
> > --- a/gcc/ChangeLog
> > +++ b/gcc/ChangeLog
> > @@ -1,3 +1,8 @@
> > +2017-02-13  Amit Pawar  <amit.pa...@amd.com>
> > +
> > + * config/i386/i386.c (znver1_cost): Fix the alignment for function
> > + and max skip bytes for function, loop and jump.
> > +
> >  2017-02-13  Martin Sebor  <mse...@redhat.com>
> >
> >   PR middle-end/79496
> > diff --git a/gcc/config/i386/i386.c b/gcc/config/i386/i386.c index
> > d7dce4b..d9a4a38 100644
> > --- a/gcc/config/i386/i386.c
> > +++ b/gcc/config/i386/i386.c
> > @@ -2672,7 +2672,7 @@ static const struct ptt
> processor_target_table[PROCESSOR_max] =
> >{"bdver4", _cost, 16, 10, 16, 7, 11},
> >{"btver1", _cost, 16, 10, 16, 7, 11},
> >{"btver2", _cost, 16, 10, 16, 7, 11},
> > -  {"znver1", _cost, 16, 10, 16, 7, 11}
> > +  {"znver1", _cost, 16, 15, 16, 15, 16}
> >  };
> >  ^L
> >  static unsigned int
> > 

Re: [PATCH] [X86_64] Fix alignment for znver1 arch.

[Uros Bizjak]

On Tue, Feb 14, 2017 at 8:48 AM, Pawar, Amit  wrote:
> Hi maintainers,
>
> Please find the below patch which changes the code alignment values for 
> znver1. Bootstrap and regression test passed on x86_64.
> OK to apply?

OK.

Thanks,
Uros.

> Thanks,
> Amit Pawar
>
> 
> diff --git a/gcc/ChangeLog b/gcc/ChangeLog
> index 2561d53..a5b0159 100644
> --- a/gcc/ChangeLog
> +++ b/gcc/ChangeLog
> @@ -1,3 +1,8 @@
> +2017-02-13  Amit Pawar  
> +
> + * config/i386/i386.c (znver1_cost): Fix the alignment for function and
> + max skip bytes for function, loop and jump.
> +
>  2017-02-13  Martin Sebor  
>
>   PR middle-end/79496
> diff --git a/gcc/config/i386/i386.c b/gcc/config/i386/i386.c
> index d7dce4b..d9a4a38 100644
> --- a/gcc/config/i386/i386.c
> +++ b/gcc/config/i386/i386.c
> @@ -2672,7 +2672,7 @@ static const struct ptt 
> processor_target_table[PROCESSOR_max] =
>{"bdver4", _cost, 16, 10, 16, 7, 11},
>{"btver1", _cost, 16, 10, 16, 7, 11},
>{"btver2", _cost, 16, 10, 16, 7, 11},
> -  {"znver1", _cost, 16, 10, 16, 7, 11}
> +  {"znver1", _cost, 16, 15, 16, 15, 16}
>  };
>  ^L
>  static unsigned int
>