On Tue, Oct 11, 2016 at 7:44 AM, Kumar, Venkataramanan <venkataramanan.ku...@amd.com> wrote: > Hi Maintainers, > > The below patch fixes integer load type reservations for -march=znver1. > > Bootstrapped and regtested on x86_64-pc-linux-gnu. > > Ok to commit to trunk ?
OK. Thanks, Uros. > (-----Snip----) > diff --git a/gcc/ChangeLog b/gcc/ChangeLog > index 9659fbf..19b4066 100644 > --- a/gcc/ChangeLog > +++ b/gcc/ChangeLog > @@ -1,3 +1,7 @@ > +2016-10-11 Venkataramanan Kumar <venkataramanan.ku...@amd.com> > + > + * config/i386/znver1.md : Fix imov/imovx load type reservations. > + > 2016-10-09 Prathamesh Kulkarni <prathamesh.kulka...@linaro.org> > > * ipa-cp.c (ipcp_alignment_lattice): Remove. > diff --git a/gcc/config/i386/znver1.md b/gcc/config/i386/znver1.md > index 7db0562..93a707a 100644 > --- a/gcc/config/i386/znver1.md > +++ b/gcc/config/i386/znver1.md > @@ -228,18 +228,18 @@ > (eq_attr "memory" "store"))) > "znver1-direct,znver1-ieu,znver1-store") > > -(define_insn_reservation "znver1_load_imov_double_load" 6 > +(define_insn_reservation "znver1_load_imov_double_load" 5 > (and (eq_attr "cpu" "znver1") > (and (eq_attr "znver1_decode" "double") > (and (eq_attr "type" "imovx") > (eq_attr "memory" "load")))) > - "znver1-double,znver1-load,znver1-ieu") > + "znver1-double,znver1-load") > > -(define_insn_reservation "znver1_load_imov_direct_load" 5 > +(define_insn_reservation "znver1_load_imov_direct_load" 4 > (and (eq_attr "cpu" "znver1") > (and (eq_attr "type" "imov,imovx") > (eq_attr "memory" "load"))) > - "znver1-direct,znver1-load,znver1-ieu") > + "znver1-direct,znver1-load") > > ;; INTEGER/GENERAL instructions > ;; register/imm operands only: ALU, ICMP, NEG, NOT, ROTATE, ISHIFT, TEST > (-----Snip----) > > Regards, > Venkat.
