Factorize vsrhrq vrshrq so that they use the same pattern. 2022-09-08 Christophe Lyon <christophe.l...@arm.com>
gcc/ * config/arm/iterators.md (MVE_VSHRQ_M_N, MVE_VSHRQ_N): New. (mve_insn): Add vrshr, vshr. * config/arm/mve.md (mve_vshrq_n_<supf><mode>) (mve_vrshrq_n_<supf><mode>): Merge into ... (@mve_<mve_insn>q_n_<supf><mode>): ... this. (mve_vrshrq_m_n_<supf><mode>, mve_vshrq_m_n_<supf><mode>): Merge into ... (@mve_<mve_insn>q_m_n_<supf><mode>): ... this. --- gcc/config/arm/iterators.md | 14 +++++++++++ gcc/config/arm/mve.md | 46 +++++++------------------------------ 2 files changed, 22 insertions(+), 38 deletions(-) diff --git a/gcc/config/arm/iterators.md b/gcc/config/arm/iterators.md index 583206dac9e..53873704174 100644 --- a/gcc/config/arm/iterators.md +++ b/gcc/config/arm/iterators.md @@ -408,6 +408,16 @@ (define_int_iterator MVE_INT_N_BINARY [ VSUBQ_N_S VSUBQ_N_U ]) +(define_int_iterator MVE_VSHRQ_M_N [ + VRSHRQ_M_N_S VRSHRQ_M_N_U + VSHRQ_M_N_S VSHRQ_M_N_U + ]) + +(define_int_iterator MVE_VSHRQ_N [ + VRSHRQ_N_S VRSHRQ_N_U + VSHRQ_N_S VSHRQ_N_U + ]) + (define_int_iterator MVE_INT_SU_N_BINARY [ VHADDQ_N_S VHADDQ_N_U VHSUBQ_N_S VHSUBQ_N_U @@ -636,6 +646,8 @@ (define_int_attr mve_insn [ (VRSHRNBQ_N_S "vrshrnb") (VRSHRNBQ_N_U "vrshrnb") (VRSHRNTQ_M_N_S "vrshrnt") (VRSHRNTQ_M_N_U "vrshrnt") (VRSHRNTQ_N_S "vrshrnt") (VRSHRNTQ_N_U "vrshrnt") + (VRSHRQ_M_N_S "vrshr") (VRSHRQ_M_N_U "vrshr") + (VRSHRQ_N_S "vrshr") (VRSHRQ_N_U "vrshr") (VSHLQ_M_N_S "vshl") (VSHLQ_M_N_U "vshl") (VSHLQ_M_R_S "vshl") (VSHLQ_M_R_U "vshl") (VSHLQ_M_S "vshl") (VSHLQ_M_U "vshl") @@ -646,6 +658,8 @@ (define_int_attr mve_insn [ (VSHRNBQ_N_S "vshrnb") (VSHRNBQ_N_U "vshrnb") (VSHRNTQ_M_N_S "vshrnt") (VSHRNTQ_M_N_U "vshrnt") (VSHRNTQ_N_S "vshrnt") (VSHRNTQ_N_U "vshrnt") + (VSHRQ_M_N_S "vshr") (VSHRQ_M_N_U "vshr") + (VSHRQ_N_S "vshr") (VSHRQ_N_U "vshr") (VSUBQ_M_N_S "vsub") (VSUBQ_M_N_U "vsub") (VSUBQ_M_N_F "vsub") (VSUBQ_M_S "vsub") (VSUBQ_M_U "vsub") (VSUBQ_M_F "vsub") (VSUBQ_N_S "vsub") (VSUBQ_N_U "vsub") (VSUBQ_N_F "vsub") diff --git a/gcc/config/arm/mve.md b/gcc/config/arm/mve.md index 20ce7ecb3d6..b5c89fd4105 100644 --- a/gcc/config/arm/mve.md +++ b/gcc/config/arm/mve.md @@ -728,18 +728,19 @@ (define_insn "@mve_<mve_insn>q_<supf><mode>" (set_attr "length""8")]) ;; -;; [vshrq_n_s, vshrq_n_u]) +;; [vrshrq_n_s, vrshrq_n_u] +;; [vshrq_n_s, vshrq_n_u] ;; ;; Version that takes an immediate as operand 2. -(define_insn "mve_vshrq_n_<supf><mode>" +(define_insn "@mve_<mve_insn>q_n_<supf><mode>" [ (set (match_operand:MVE_2 0 "s_register_operand" "=w") (unspec:MVE_2 [(match_operand:MVE_2 1 "s_register_operand" "w") (match_operand:SI 2 "<MVE_pred2>" "<MVE_constraint2>")] - VSHRQ_N)) + MVE_VSHRQ_N)) ] "TARGET_HAVE_MVE" - "vshr.<supf><V_sz_elem>\t%q0, %q1, %2" + "<mve_insn>.<supf><V_sz_elem>\t%q0, %q1, %2" [(set_attr "type" "mve_move") ]) @@ -1401,21 +1402,6 @@ (define_insn "mve_vqshluq_n_s<mode>" [(set_attr "type" "mve_move") ]) -;; -;; [vrshrq_n_s, vrshrq_n_u]) -;; -(define_insn "mve_vrshrq_n_<supf><mode>" - [ - (set (match_operand:MVE_2 0 "s_register_operand" "=w") - (unspec:MVE_2 [(match_operand:MVE_2 1 "s_register_operand" "w") - (match_operand:SI 2 "<MVE_pred2>" "<MVE_constraint2>")] - VRSHRQ_N)) - ] - "TARGET_HAVE_MVE" - "vrshr.<supf>%#<V_sz_elem>\t%q0, %q1, %2" - [(set_attr "type" "mve_move") -]) - ;; ;; [vabdq_f] ;; @@ -4661,35 +4647,19 @@ (define_insn "@mve_<mve_insn>q_m_n_<supf><mode>" ;; ;; [vrshrq_m_n_s, vrshrq_m_n_u]) -;; -(define_insn "mve_vrshrq_m_n_<supf><mode>" - [ - (set (match_operand:MVE_2 0 "s_register_operand" "=w") - (unspec:MVE_2 [(match_operand:MVE_2 1 "s_register_operand" "0") - (match_operand:MVE_2 2 "s_register_operand" "w") - (match_operand:SI 3 "<MVE_pred2>" "<MVE_constraint2>") - (match_operand:<MVE_VPRED> 4 "vpr_register_operand" "Up")] - VRSHRQ_M_N)) - ] - "TARGET_HAVE_MVE" - "vpst\;vrshrt.<supf>%#<V_sz_elem>\t%q0, %q2, %3" - [(set_attr "type" "mve_move") - (set_attr "length""8")]) - -;; ;; [vshrq_m_n_s, vshrq_m_n_u]) ;; -(define_insn "mve_vshrq_m_n_<supf><mode>" +(define_insn "@mve_<mve_insn>q_m_n_<supf><mode>" [ (set (match_operand:MVE_2 0 "s_register_operand" "=w") (unspec:MVE_2 [(match_operand:MVE_2 1 "s_register_operand" "0") (match_operand:MVE_2 2 "s_register_operand" "w") (match_operand:SI 3 "<MVE_pred2>" "<MVE_constraint2>") (match_operand:<MVE_VPRED> 4 "vpr_register_operand" "Up")] - VSHRQ_M_N)) + MVE_VSHRQ_M_N)) ] "TARGET_HAVE_MVE" - "vpst\;vshrt.<supf>%#<V_sz_elem>\t%q0, %q2, %3" + "vpst\;<mve_insn>t.<supf>%#<V_sz_elem>\t%q0, %q2, %3" [(set_attr "type" "mve_move") (set_attr "length""8")]) -- 2.34.1