From a39d4348cdf4a5521710389143b93d9047e23a46 Mon Sep 17 00:00:00 2001 From: James Greenhalgh Date: Wed, 26 Jul 2017 17:40:39 +0000 Subject: [PATCH] [Patch AArch64 obvious] Unify address costs to generic_addrcost_table 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 | 9 ++++++++ gcc/config/aarch64/aarch64.c | 40 ++++-------------------------------- 2 files changed, 13 insertions(+), 36 deletions(-) diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 188b0f3bbd2..aa8e9e3f765 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,12 @@ +2017-07-26 James Greenhalgh + + * 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 * config/aarch64/aarch64.c (cortexa57_branch_cost): Remove. diff --git a/gcc/config/aarch64/aarch64.c b/gcc/config/aarch64/aarch64.c index 31c0d8b6920..9aa59e7fe62 100644 --- a/gcc/config/aarch64/aarch64.c +++ b/gcc/config/aarch64/aarch64.c @@ -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, -- 2.30.2