On Tue, Dec 15, 2015 at 10:59 AM, Wilco Dijkstra <wilco.dijks...@arm.com> wrote: > ping > >> -----Original Message----- >> From: Wilco Dijkstra [mailto:wilco.dijks...@arm.com] >> Sent: 19 November 2015 18:12 >> To: gcc-patches@gcc.gnu.org >> Subject: [PATCH][ARM] Enable fusion of AES instructions >> >> Enable instruction fusion of AES instructions on ARM for Cortex-A53 and >> Cortex-A57. >> >> OK for commit? >> >> ChangeLog: >> 2015-11-20 Wilco Dijkstra <wdijk...@arm.com> >> >> * gcc/config/arm/arm.c (arm_cortex_a53_tune): Add AES fusion. >> (arm_cortex_a57_tune): Likewise. >> (aarch_macro_fusion_pair_p): Add support for AES fusion. >> * gcc/config/arm/arm-protos.h (fuse_ops): Add FUSE_AES_AESMC. >> >> --- >> gcc/config/arm/arm-protos.h | 5 +++-- >> gcc/config/arm/arm.c | 9 +++++++-- >> 2 files changed, 10 insertions(+), 4 deletions(-) >> >> diff --git a/gcc/config/arm/arm-protos.h b/gcc/config/arm/arm-protos.h >> index f9b1276..4801bb8 100644 >> --- a/gcc/config/arm/arm-protos.h >> +++ b/gcc/config/arm/arm-protos.h >> @@ -302,8 +302,9 @@ struct tune_params >> enum fuse_ops >> { >> FUSE_NOTHING = 0, >> - FUSE_MOVW_MOVT = 1 << 0 >> - } fusible_ops: 1; >> + FUSE_MOVW_MOVT = 1 << 0, >> + FUSE_AES_AESMC = 1 << 1 >> + } fusible_ops: 2; >> /* Depth of scheduling queue to check for L2 autoprefetcher. */ >> enum {SCHED_AUTOPREF_OFF, SCHED_AUTOPREF_RANK, SCHED_AUTOPREF_FULL} >> sched_autopref: 2; >> diff --git a/gcc/config/arm/arm.c b/gcc/config/arm/arm.c >> index 02f5dc3..7077199 100644 >> --- a/gcc/config/arm/arm.c >> +++ b/gcc/config/arm/arm.c >> @@ -1969,7 +1969,7 @@ const struct tune_params arm_cortex_a53_tune = >> tune_params::DISPARAGE_FLAGS_NEITHER, >> tune_params::PREF_NEON_64_FALSE, >> tune_params::PREF_NEON_STRINGOPS_TRUE, >> - FUSE_OPS (tune_params::FUSE_MOVW_MOVT), >> + FUSE_OPS (tune_params::FUSE_MOVW_MOVT | tune_params::FUSE_AES_AESMC), >> tune_params::SCHED_AUTOPREF_OFF >> }; >> >> @@ -1992,7 +1992,7 @@ const struct tune_params arm_cortex_a57_tune = >> tune_params::DISPARAGE_FLAGS_ALL, >> tune_params::PREF_NEON_64_FALSE, >> tune_params::PREF_NEON_STRINGOPS_TRUE, >> - FUSE_OPS (tune_params::FUSE_MOVW_MOVT), >> + FUSE_OPS (tune_params::FUSE_MOVW_MOVT | tune_params::FUSE_AES_AESMC), >> tune_params::SCHED_AUTOPREF_FULL >> }; >> >> @@ -29668,6 +29668,11 @@ aarch_macro_fusion_pair_p (rtx_insn* prev, >> rtx_insn* curr) >> && REGNO (SET_DEST (curr_set)) == REGNO (SET_DEST (prev_set))) >> return true; >> } >> + >> + if (current_tune->fusible_ops & tune_params::FUSE_AES_AESMC >> + && aarch_crypto_can_dual_issue (prev, curr)) >> + return true; >> + >> return false; >> }
OK - this is purely a core tuning patch and thus is acceptable even at this stage. regards Ramana >> >> -- >> 1.9.1 >
