diff options
author | Ju-Zhe Zhong <juzhe.zhong@rivai.ai> | 2023-03-08 15:42:13 +0800 |
---|---|---|
committer | Kito Cheng <kito.cheng@sifive.com> | 2023-03-10 16:25:03 +0800 |
commit | a803c268c5529624bdb7d02131d4862516a63c22 (patch) | |
tree | aec3f861446e1c9de51cec399463f1f0406a9db5 /gcc | |
parent | ab7bb445ee586258a6210462e92ed196d61beb9e (diff) | |
download | gcc-a803c268c5529624bdb7d02131d4862516a63c22.zip gcc-a803c268c5529624bdb7d02131d4862516a63c22.tar.gz gcc-a803c268c5529624bdb7d02131d4862516a63c22.tar.bz2 |
Extend nops num in "maybe_gen_insn" for RISC-V Vector intrinsics
Hi, current maybe_gen_insn can only expand 9 nops.
For RVV intrinsics, I need to extend it as 10, otherwise I should use GEN_FCN.
This patch is quite obvious change, Ok for trunk ?
Thanks.
gcc/ChangeLog:
* config/riscv/riscv-vector-builtins.cc
(function_expander::use_ternop_insn): Use maybe_gen_insn instead.
(function_expander::use_widen_ternop_insn): Ditto.
* optabs.cc (maybe_gen_insn): Extend nops handling.
Diffstat (limited to 'gcc')
-rw-r--r-- | gcc/config/riscv/riscv-vector-builtins.cc | 24 | ||||
-rw-r--r-- | gcc/optabs.cc | 5 |
2 files changed, 7 insertions, 22 deletions
diff --git a/gcc/config/riscv/riscv-vector-builtins.cc b/gcc/config/riscv/riscv-vector-builtins.cc index 2d57086..680c165 100644 --- a/gcc/config/riscv/riscv-vector-builtins.cc +++ b/gcc/config/riscv/riscv-vector-builtins.cc @@ -3100,17 +3100,7 @@ function_expander::use_ternop_insn (bool vd_accum_p, insn_code icode) add_input_operand (Pmode, get_tail_policy_for_pred (pred)); add_input_operand (Pmode, get_mask_policy_for_pred (pred)); add_input_operand (Pmode, get_avl_type_rtx (avl_type::NONVLMAX)); - - /* See optabs.cc, the maximum nops is 9 for using 'maybe_gen_insn'. - We temporarily use GCN directly. We will change it back it we - can support nops >= 10. */ - gcc_assert (maybe_legitimize_operands (icode, 0, opno, m_ops)); - rtx_insn *pat = GEN_FCN ( - icode) (m_ops[0].value, m_ops[1].value, m_ops[2].value, m_ops[3].value, - m_ops[4].value, m_ops[5].value, m_ops[6].value, m_ops[7].value, - m_ops[8].value, m_ops[9].value); - emit_insn (pat); - return m_ops[0].value; + return generate_insn (icode); } /* Implement the call using instruction ICODE, with a 1:1 mapping between @@ -3142,17 +3132,7 @@ function_expander::use_widen_ternop_insn (insn_code icode) add_input_operand (Pmode, get_tail_policy_for_pred (pred)); add_input_operand (Pmode, get_mask_policy_for_pred (pred)); add_input_operand (Pmode, get_avl_type_rtx (avl_type::NONVLMAX)); - - /* See optabs.cc, the maximum nops is 9 for using 'maybe_gen_insn'. - We temporarily use GCN directly. We will change it back it we - can support nops >= 10. */ - gcc_assert (maybe_legitimize_operands (icode, 0, opno, m_ops)); - rtx_insn *pat = GEN_FCN ( - icode) (m_ops[0].value, m_ops[1].value, m_ops[2].value, m_ops[3].value, - m_ops[4].value, m_ops[5].value, m_ops[6].value, m_ops[7].value, - m_ops[8].value, m_ops[9].value); - emit_insn (pat); - return m_ops[0].value; + return generate_insn (icode); } /* Implement the call using instruction ICODE, with a 1:1 mapping between diff --git a/gcc/optabs.cc b/gcc/optabs.cc index cf22bfe..4c641ca 100644 --- a/gcc/optabs.cc +++ b/gcc/optabs.cc @@ -8091,6 +8091,11 @@ maybe_gen_insn (enum insn_code icode, unsigned int nops, return GEN_FCN (icode) (ops[0].value, ops[1].value, ops[2].value, ops[3].value, ops[4].value, ops[5].value, ops[6].value, ops[7].value, ops[8].value); + case 10: + return GEN_FCN (icode) (ops[0].value, ops[1].value, ops[2].value, + ops[3].value, ops[4].value, ops[5].value, + ops[6].value, ops[7].value, ops[8].value, + ops[9].value); } gcc_unreachable (); } |