diff mbox

[x86_64] Fix znver1 imov/imovx load reservations.

Message ID CY4PR12MB1736E17964F276935F17AE788FDA0@CY4PR12MB1736.namprd12.prod.outlook.com
State New
Headers show

Commit Message

Kumar, Venkataramanan Oct. 11, 2016, 5:44 a.m. UTC 
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 ? 

(-----Snip----)
(-----Snip----)

Regards,
Venkat.

Comments

Uros Bizjak Oct. 11, 2016, 6:48 a.m. UTC | #1 
On Tue, Oct 11, 2016 at 7:44 AM, Kumar, Venkataramanan
<Venkataramanan.Kumar@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.kumar@amd.com>
> +
> +       * config/i386/znver1.md : Fix imov/imovx load type reservations.
> +
>  2016-10-09  Prathamesh Kulkarni  <prathamesh.kulkarni@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.
diff mbox

Patch

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.kumar@amd.com>
+
+       * config/i386/znver1.md : Fix imov/imovx load type reservations.
+
 2016-10-09  Prathamesh Kulkarni  <prathamesh.kulkarni@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