[Patch AArch64 obvious] Unify address costs to generic_addrcost_table
authorJames Greenhalgh <james.greenhalgh@arm.com>
Wed, 26 Jul 2017 17:40:39 +0000 (17:40 +0000)
committerJames Greenhalgh <jgreenhalgh@gcc.gnu.org>
Wed, 26 Jul 2017 17:40:39 +0000 (17:40 +0000)
The special case address cost tables for Cortex-A57 and qdf24xx are no
different from the generic address cost table. We should just use the
address cost table directly. If this changes in future, a core is welcome
to add new address cost tables.

gcc/

* config/aarch64/aarch64.c (cortexa57_addrcost_table): Remove.
(qdf24xx_addrcost_table): Likewise.
(cortexa57_tunings): Update to use generic_branch_cost.
(cortexa72_tunings): Likewise.
(cortexa73_tunings): Likewise.
(qdf24xx_tunings): Likewise.

From-SVN: r250585

gcc/ChangeLog
gcc/config/aarch64/aarch64.c

index 188b0f3bbd2a01490c14db1d7a9edc91f9c66889..aa8e9e3f7658a0687309019297e5ab0443f7d2ac 100644 (file)
@@ -1,3 +1,12 @@
+2017-07-26  James Greenhalgh  <james.greenhalgh@arm.com>
+
+       * config/aarch64/aarch64.c (cortexa57_addrcost_table): Remove.
+       (qdf24xx_addrcost_table): Likewise.
+       (cortexa57_tunings): Update to use generic_branch_cost.
+       (cortexa72_tunings): Likewise.
+       (cortexa73_tunings): Likewise.
+       (qdf24xx_tunings): Likewise.
+
 2017-07-26  James Greenhalgh  <james.greenhalgh@arm.com>
 
        * config/aarch64/aarch64.c (cortexa57_branch_cost): Remove.
index 31c0d8b692099b3d94110b4c869f5fa21663818e..9aa59e7fe62aa6b7e1e1e3ac46d8aaa93655165f 100644 (file)
@@ -206,22 +206,6 @@ static const struct cpu_addrcost_table generic_addrcost_table =
   0 /* imm_offset  */
 };
 
-static const struct cpu_addrcost_table cortexa57_addrcost_table =
-{
-    {
-      1, /* hi  */
-      0, /* si  */
-      0, /* di  */
-      1, /* ti  */
-    },
-  0, /* pre_modify  */
-  0, /* post_modify  */
-  0, /* register_offset  */
-  0, /* register_sextend  */
-  0, /* register_zextend  */
-  0, /* imm_offset  */
-};
-
 static const struct cpu_addrcost_table exynosm1_addrcost_table =
 {
     {
@@ -254,22 +238,6 @@ static const struct cpu_addrcost_table xgene1_addrcost_table =
   0, /* imm_offset  */
 };
 
-static const struct cpu_addrcost_table qdf24xx_addrcost_table =
-{
-    {
-      1, /* hi  */
-      0, /* si  */
-      0, /* di  */
-      1, /* ti  */
-    },
-  0, /* pre_modify  */
-  0, /* post_modify  */
-  0, /* register_offset  */
-  0, /* register_sextend  */
-  0, /* register_zextend  */
-  0 /* imm_offset  */
-};
-
 static const struct cpu_addrcost_table thunderx2t99_addrcost_table =
 {
     {
@@ -647,7 +615,7 @@ static const struct tune_params cortexa53_tunings =
 static const struct tune_params cortexa57_tunings =
 {
   &cortexa57_extra_costs,
-  &cortexa57_addrcost_table,
+  &generic_addrcost_table,
   &cortexa57_regmove_cost,
   &cortexa57_vector_cost,
   &generic_branch_cost,
@@ -673,7 +641,7 @@ static const struct tune_params cortexa57_tunings =
 static const struct tune_params cortexa72_tunings =
 {
   &cortexa57_extra_costs,
-  &cortexa57_addrcost_table,
+  &generic_addrcost_table,
   &cortexa57_regmove_cost,
   &cortexa57_vector_cost,
   &generic_branch_cost,
@@ -699,7 +667,7 @@ static const struct tune_params cortexa72_tunings =
 static const struct tune_params cortexa73_tunings =
 {
   &cortexa57_extra_costs,
-  &cortexa57_addrcost_table,
+  &generic_addrcost_table,
   &cortexa57_regmove_cost,
   &cortexa57_vector_cost,
   &generic_branch_cost,
@@ -828,7 +796,7 @@ static const struct tune_params xgene1_tunings =
 static const struct tune_params qdf24xx_tunings =
 {
   &qdf24xx_extra_costs,
-  &qdf24xx_addrcost_table,
+  &generic_addrcost_table,
   &qdf24xx_regmove_cost,
   &generic_vector_cost,
   &generic_branch_cost,