[PATCH] [X86_64] Fix alignment for znver1 arch.
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? 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", &bdver4_cost, 16, 10, 16, 7, 11}, {"btver1", &btver1_cost, 16, 10, 16, 7, 11}, {"btver2", &btver2_cost, 16, 10, 16, 7, 11}, - {"znver1", &znver1_cost, 16, 10, 16, 7, 11} + {"znver1", &znver1_cost, 16, 15, 16, 15, 16} }; ^L static unsigned int
Re: [PATCH] [X86_64] Fix alignment for znver1 arch.
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", &bdver4_cost, 16, 10, 16, 7, 11}, >{"btver1", &btver1_cost, 16, 10, 16, 7, 11}, >{"btver2", &btver2_cost, 16, 10, 16, 7, 11}, > - {"znver1", &znver1_cost, 16, 10, 16, 7, 11} > + {"znver1", &znver1_cost, 16, 15, 16, 15, 16} > }; > ^L > static unsigned int >
RE: [PATCH] [X86_64] Fix alignment for znver1 arch.
Thanks Uros, I committed on Amit's behalf. https://gcc.gnu.org/viewcvs/gcc?view=revision&revision=245423 regards, venkat. > -Original Message- > From: Uros Bizjak [mailto:ubiz...@gmail.com] > Sent: Tuesday, February 14, 2017 2:43 PM > To: Pawar, Amit > Cc: gcc-patches@gcc.gnu.org; Kumar, Venkataramanan > > Subject: Re: [PATCH] [X86_64] Fix alignment for znver1 arch. > > 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", &bdver4_cost, 16, 10, 16, 7, 11}, > >{"btver1", &btver1_cost, 16, 10, 16, 7, 11}, > >{"btver2", &btver2_cost, 16, 10, 16, 7, 11}, > > - {"znver1", &znver1_cost, 16, 10, 16, 7, 11} > > + {"znver1", &znver1_cost, 16, 15, 16, 15, 16} > > }; > > ^L > > static unsigned int > >