From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 7928) id 2994F3861829; Thu, 21 Sep 2023 10:12:53 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 2994F3861829 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gcc.gnu.org; s=default; t=1695291173; bh=BoNkjHw7x/Vgk40BlrF0sjMp+CIMm8bvspVxpUyJ09A=; h=From:To:Subject:Date:From; b=aP6gUvtOooBAURnyHIz9HIHbTQUayjyDB3HhiRNhYj4U5XT6OwJSMQqCaDPBTNc8O 9ZvmktEALtRV+RsYLtsX4SQBwPO6S6PwESQ+klZFDW2nONEmPxUN2zavMmMWxlW3rt ycyb3YggUyo8SACGGjFMThCTZH6FyvKCenYjwruk= MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="utf-8" From: Lehua Ding To: gcc-cvs@gcc.gnu.org Subject: [gcc r14-4209] RISC-V: Adjusting the comments of the emit_vlmax_insn/emit_vlmax_insn_lra/emit_nonvlmax_insn functio X-Act-Checkin: gcc X-Git-Author: Lehua Ding X-Git-Refname: refs/heads/trunk X-Git-Oldrev: 7439f40dc17f47480620beadf5b2f5784b59aefe X-Git-Newrev: 5ff4431675c0d0c800d4a983254e94a6b401c14d Message-Id: <20230921101253.2994F3861829@sourceware.org> Date: Thu, 21 Sep 2023 10:12:53 +0000 (GMT) List-Id: https://gcc.gnu.org/g:5ff4431675c0d0c800d4a983254e94a6b401c14d commit r14-4209-g5ff4431675c0d0c800d4a983254e94a6b401c14d Author: Lehua Ding Date: Thu Sep 21 15:02:32 2023 +0800 RISC-V: Adjusting the comments of the emit_vlmax_insn/emit_vlmax_insn_lra/emit_nonvlmax_insn functions V2 Change: Use Robin's comments. This patch adjusts the comments of the emit_vlmax_insn/emit_vlmax_insn_lra/emit_nonvlmax_insn functions. The purpose of the adjustment is to make it clear that vlmax here is not VLMAX as defined inside the RVV ISA. This is because this function is used by RVV mode (e.g. RVVM1SImode) in addition to VLS mode (V16QI). For RVV mode, it means the same thing, for VLS mode, it indicates setting the vl to the number of units of the mode. Changed the comment because I didn't think of a better name. If there is a suitable name, feel free to discuss it. gcc/ChangeLog: * config/riscv/riscv-v.cc (emit_vlmax_insn): Adjust comments. (emit_nonvlmax_insn): Adjust comments. (emit_vlmax_insn_lra): Adjust comments. Co-Authored-By: Robin Dapp Diff: --- gcc/config/riscv/riscv-v.cc | 33 +++++++++++++++++++++------------ 1 file changed, 21 insertions(+), 12 deletions(-) diff --git a/gcc/config/riscv/riscv-v.cc b/gcc/config/riscv/riscv-v.cc index 366f0659817..4b9a494f8eb 100644 --- a/gcc/config/riscv/riscv-v.cc +++ b/gcc/config/riscv/riscv-v.cc @@ -347,33 +347,42 @@ private: expand_operand m_ops[MAX_OPERANDS]; }; -/* Emit RVV insn which vl is VLMAX. - This function can only be used before LRA pass or - for VLS_AVL_IMM modes. */ +/* Emit an RVV insn with a vector length that equals the number of units of the + vector mode. For VLA modes this corresponds to VLMAX. + + Unless the vector length can be encoded in the vsetivl[i] instruction this + function must only be used as long as we can create pseudo registers. This is + because it will set a pseudo register to VLMAX using vsetvl and use this as + definition for the vector length. */ void emit_vlmax_insn (unsigned icode, unsigned insn_flags, rtx *ops) { insn_expander e (insn_flags, true); + gcc_assert (can_create_pseudo_p () || const_vlmax_p (e.get_vtype_mode (ops))); + e.emit_insn ((enum insn_code) icode, ops); } -/* Emit RVV insn which vl is VL. */ +/* Like emit_vlmax_insn but must only be used when we cannot create pseudo + registers anymore. This function, however, takes a predefined vector length + from the value in VL. */ void -emit_nonvlmax_insn (unsigned icode, unsigned insn_flags, rtx *ops, rtx vl) +emit_vlmax_insn_lra (unsigned icode, unsigned insn_flags, rtx *ops, rtx vl) { - insn_expander e (insn_flags, false); + gcc_assert (!can_create_pseudo_p ()); + + insn_expander e (insn_flags, true); e.set_vl (vl); e.emit_insn ((enum insn_code) icode, ops); } -/* Emit RVV insn which vl is VL but the AVL_TYPE insn attr is VLMAX. - This function used after LRA pass that cann't create pseudo register. */ +/* Emit an RVV insn with a predefined vector length. Contrary to + emit_vlmax_insn the instruction's vector length is not deduced from its mode + but taken from the value in VL. */ void -emit_vlmax_insn_lra (unsigned icode, unsigned insn_flags, rtx *ops, rtx vl) +emit_nonvlmax_insn (unsigned icode, unsigned insn_flags, rtx *ops, rtx vl) { - gcc_assert (!can_create_pseudo_p ()); - - insn_expander e (insn_flags, true); + insn_expander e (insn_flags, false); e.set_vl (vl); e.emit_insn ((enum insn_code) icode, ops); }