RISC-V: Remove autovec-vls.md file and clean up VLS move modes[NFC]

Message ID 20230918070713.3569601-1-juzhe.zhong@rivai.ai
State Unresolved
Headers
Series RISC-V: Remove autovec-vls.md file and clean up VLS move modes[NFC] |

Checks

Context Check Description
snail/gcc-patch-check warning Git am fail log

Commit Message

juzhe.zhong@rivai.ai Sept. 18, 2023, 7:07 a.m. UTC
  We have largely supportted VLS modes. Only move patterns of VLS modes are
different from VLS patterns. The rest of them are the same.

We always extend the current VLA patterns with VLSmodes:

VI --> V_VLSI
VF --> V_VLSF

It makes no sense to have a separate file holding a very few VLS patterns
that can not be extended from the current VLA patterns.

So remove autovec-vls.md

gcc/ChangeLog:

	* config/riscv/vector.md (mov<mode>): New pattern.
	(*mov<mode>_mem_to_mem): Ditto.
	(*mov<mode>): Ditto.
	(@mov<VLS_AVL_REG:mode><P:mode>_lra): Ditto.
	(*mov<VLS_AVL_REG:mode><P:mode>_lra): Ditto.
	(*mov<mode>_vls): Ditto.
	(movmisalign<mode>): Ditto.
	(@vec_duplicate<mode>): Ditto.
	* config/riscv/autovec-vls.md: Removed.

---
 gcc/config/riscv/autovec-vls.md | 196 --------------------------------
 gcc/config/riscv/vector.md      | 172 +++++++++++++++++++++++++++-
 2 files changed, 170 insertions(+), 198 deletions(-)
 delete mode 100644 gcc/config/riscv/autovec-vls.md
  

Comments

Kito Cheng Sept. 18, 2023, 8:01 a.m. UTC | #1
LGTM :)

On Mon, Sep 18, 2023 at 3:07 PM Juzhe-Zhong <juzhe.zhong@rivai.ai> wrote:
>
> We have largely supportted VLS modes. Only move patterns of VLS modes are
> different from VLS patterns. The rest of them are the same.
>
> We always extend the current VLA patterns with VLSmodes:
>
> VI --> V_VLSI
> VF --> V_VLSF
>
> It makes no sense to have a separate file holding a very few VLS patterns
> that can not be extended from the current VLA patterns.
>
> So remove autovec-vls.md
>
> gcc/ChangeLog:
>
>         * config/riscv/vector.md (mov<mode>): New pattern.
>         (*mov<mode>_mem_to_mem): Ditto.
>         (*mov<mode>): Ditto.
>         (@mov<VLS_AVL_REG:mode><P:mode>_lra): Ditto.
>         (*mov<VLS_AVL_REG:mode><P:mode>_lra): Ditto.
>         (*mov<mode>_vls): Ditto.
>         (movmisalign<mode>): Ditto.
>         (@vec_duplicate<mode>): Ditto.
>         * config/riscv/autovec-vls.md: Removed.
>
> ---
>  gcc/config/riscv/autovec-vls.md | 196 --------------------------------
>  gcc/config/riscv/vector.md      | 172 +++++++++++++++++++++++++++-
>  2 files changed, 170 insertions(+), 198 deletions(-)
>  delete mode 100644 gcc/config/riscv/autovec-vls.md
>
> diff --git a/gcc/config/riscv/autovec-vls.md b/gcc/config/riscv/autovec-vls.md
> deleted file mode 100644
> index 3488f452e5d..00000000000
> --- a/gcc/config/riscv/autovec-vls.md
> +++ /dev/null
> @@ -1,196 +0,0 @@
> -;; Machine description for VLS of RVV auto-vectorization.
> -;; Copyright (C) 2023 Free Software Foundation, Inc.
> -;; Contributed by Juzhe Zhong (juzhe.zhong@rivai.ai), RiVAI Technologies Ltd.
> -
> -;; This file is part of GCC.
> -
> -;; GCC is free software; you can redistribute it and/or modify
> -;; it under the terms of the GNU General Public License as published by
> -;; the Free Software Foundation; either version 3, or (at your option)
> -;; any later version.
> -
> -;; GCC is distributed in the hope that it will be useful,
> -;; but WITHOUT ANY WARRANTY; without even the implied warranty of
> -;; MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> -;; GNU General Public License for more details.
> -
> -;; You should have received a copy of the GNU General Public License
> -;; along with GCC; see the file COPYING3.  If not see
> -;; <http://www.gnu.org/licenses/>.
> -
> -;; We define VLS modes as 'define_insn_and_split' with normal
> -;; RTX_CODE operation, so we can gain benefits from Combine optimizations.
> -
> -;; -----------------------------------------------------------------
> -;; ---- Moves Operations
> -;; -----------------------------------------------------------------
> -
> -(define_expand "mov<mode>"
> -  [(set (match_operand:VLS_AVL_IMM 0 "reg_or_mem_operand")
> -       (match_operand:VLS_AVL_IMM 1 "general_operand"))]
> -  "TARGET_VECTOR"
> -{
> -  if (riscv_vector::legitimize_move (operands[0], operands[1]))
> -    DONE;
> -})
> -
> -(define_insn_and_split "*mov<mode>_mem_to_mem"
> -  [(set (match_operand:VLS_AVL_IMM 0 "memory_operand")
> -       (match_operand:VLS_AVL_IMM 1 "memory_operand"))]
> -  "TARGET_VECTOR && can_create_pseudo_p ()"
> -  "#"
> -  "&& 1"
> -  [(const_int 0)]
> -  {
> -    if (GET_MODE_BITSIZE (<MODE>mode).to_constant () <= MAX_BITS_PER_WORD)
> -      {
> -        /* Opitmize the following case:
> -
> -           typedef int8_t v2qi __attribute__ ((vector_size (2)));
> -           v2qi v = *(v2qi*)in;
> -           *(v2qi*)out = v;
> -
> -           We prefer scalar load/store instead of vle.v/vse.v when
> -           the VLS modes size is smaller scalar mode.  */
> -        machine_mode mode;
> -        unsigned size = GET_MODE_BITSIZE (<MODE>mode).to_constant ();
> -        if (FLOAT_MODE_P (<MODE>mode))
> -         mode = mode_for_size (size, MODE_FLOAT, 0).require ();
> -        else
> -         mode = mode_for_size (size, MODE_INT, 0).require ();
> -        emit_move_insn (gen_lowpart (mode, operands[0]),
> -                       gen_lowpart (mode, operands[1]));
> -      }
> -    else
> -      {
> -       operands[1] = force_reg (<MODE>mode, operands[1]);
> -       emit_move_insn (operands[0], operands[1]);
> -      }
> -    DONE;
> -  }
> -  [(set_attr "type" "vmov")]
> -)
> -
> -(define_insn_and_split "*mov<mode>"
> -  [(set (match_operand:VLS_AVL_IMM 0 "reg_or_mem_operand" "=vr, m, vr")
> -       (match_operand:VLS_AVL_IMM 1 "reg_or_mem_operand" "  m,vr, vr"))]
> -  "TARGET_VECTOR
> -   && (register_operand (operands[0], <MODE>mode)
> -       || register_operand (operands[1], <MODE>mode))"
> -  "@
> -   #
> -   #
> -   vmv%m1r.v\t%0,%1"
> -  "&& reload_completed
> -   && (!register_operand (operands[0], <MODE>mode)
> -       || !register_operand (operands[1], <MODE>mode))"
> -  [(const_int 0)]
> -  {
> -    bool ok_p = riscv_vector::legitimize_move (operands[0], operands[1]);
> -    gcc_assert (ok_p);
> -    DONE;
> -  }
> -  [(set_attr "type" "vmov")]
> -)
> -
> -(define_expand "mov<mode>"
> -  [(set (match_operand:VLS_AVL_REG 0 "reg_or_mem_operand")
> -       (match_operand:VLS_AVL_REG 1 "general_operand"))]
> -  "TARGET_VECTOR"
> -{
> -  bool ok_p = riscv_vector::legitimize_move (operands[0], operands[1]);
> -  gcc_assert (ok_p);
> -  DONE;
> -})
> -
> -(define_expand "@mov<VLS_AVL_REG:mode><P:mode>_lra"
> -  [(parallel
> -    [(set (match_operand:VLS_AVL_REG 0 "reg_or_mem_operand")
> -         (match_operand:VLS_AVL_REG 1 "reg_or_mem_operand"))
> -   (clobber (match_scratch:P 2))])]
> -  "TARGET_VECTOR && (lra_in_progress || reload_completed)"
> -{})
> -
> -(define_insn_and_split "*mov<VLS_AVL_REG:mode><P:mode>_lra"
> -  [(set (match_operand:VLS_AVL_REG 0 "reg_or_mem_operand" "=vr, m,vr")
> -       (match_operand:VLS_AVL_REG 1 "reg_or_mem_operand" "  m,vr,vr"))
> -   (clobber (match_scratch:P 2 "=&r,&r,X"))]
> -  "TARGET_VECTOR && (lra_in_progress || reload_completed)
> -   && (register_operand (operands[0], <VLS_AVL_REG:MODE>mode)
> -       || register_operand (operands[1], <VLS_AVL_REG:MODE>mode))"
> -  "#"
> -  "&& reload_completed"
> -  [(const_int 0)]
> -{
> -  if (REG_P (operands[0]) && REG_P (operands[1]))
> -      emit_insn (gen_rtx_SET (operands[0], operands[1]));
> -  else
> -    {
> -      emit_move_insn (operands[2], gen_int_mode (GET_MODE_NUNITS (<VLS_AVL_REG:MODE>mode),
> -                                                Pmode));
> -      unsigned insn_flags
> -        = GET_MODE_CLASS (<VLS_AVL_REG:MODE>mode) == MODE_VECTOR_BOOL
> -                                                    ? riscv_vector::UNARY_MASK_OP
> -                                                    : riscv_vector::UNARY_OP;
> -      riscv_vector::emit_nonvlmax_insn (code_for_pred_mov (<VLS_AVL_REG:MODE>mode),
> -                                       insn_flags, operands, operands[2]);
> -    }
> -  DONE;
> -}
> -  [(set_attr "type" "vmov")]
> -)
> -
> -(define_insn "*mov<mode>_vls"
> -  [(set (match_operand:VLS 0 "register_operand" "=vr")
> -       (match_operand:VLS 1 "register_operand" " vr"))]
> -  "TARGET_VECTOR"
> -  "vmv%m1r.v\t%0,%1"
> -  [(set_attr "type" "vmov")
> -   (set_attr "mode" "<MODE>")])
> -
> -(define_insn "*mov<mode>_vls"
> -  [(set (match_operand:VLSB 0 "register_operand" "=vr")
> -       (match_operand:VLSB 1 "register_operand" " vr"))]
> -  "TARGET_VECTOR"
> -  "vmv1r.v\t%0,%1"
> -  [(set_attr "type" "vmov")
> -   (set_attr "mode" "<MODE>")])
> -
> -(define_expand "movmisalign<mode>"
> -  [(set (match_operand:VLS 0 "nonimmediate_operand")
> -       (match_operand:VLS 1 "general_operand"))]
> -  "TARGET_VECTOR"
> -  {
> -    /* To support misalign data movement, we should use
> -       minimum element alignment load/store.  */
> -    unsigned int size = GET_MODE_SIZE (GET_MODE_INNER (<MODE>mode));
> -    poly_int64 nunits = GET_MODE_NUNITS (<MODE>mode) * size;
> -    machine_mode mode = riscv_vector::get_vector_mode (QImode, nunits).require ();
> -    operands[0] = gen_lowpart (mode, operands[0]);
> -    operands[1] = gen_lowpart (mode, operands[1]);
> -    if (MEM_P (operands[0]) && !register_operand (operands[1], mode))
> -      operands[1] = force_reg (mode, operands[1]);
> -    riscv_vector::emit_vlmax_insn (code_for_pred_mov (mode), riscv_vector::UNARY_OP, operands);
> -    DONE;
> -  }
> -)
> -
> -;; -----------------------------------------------------------------
> -;; ---- Duplicate Operations
> -;; -----------------------------------------------------------------
> -
> -(define_insn_and_split "@vec_duplicate<mode>"
> -  [(set (match_operand:VLS 0 "register_operand")
> -        (vec_duplicate:VLS
> -          (match_operand:<VEL> 1 "reg_or_int_operand")))]
> -  "TARGET_VECTOR && can_create_pseudo_p ()"
> -  "#"
> -  "&& 1"
> -  [(const_int 0)]
> -  {
> -    riscv_vector::emit_vlmax_insn (code_for_pred_broadcast (<MODE>mode),
> -                                   riscv_vector::UNARY_OP, operands);
> -    DONE;
> -  }
> -  [(set_attr "type" "vector")]
> -)
> diff --git a/gcc/config/riscv/vector.md b/gcc/config/riscv/vector.md
> index 939e992cad1..40512e8d864 100644
> --- a/gcc/config/riscv/vector.md
> +++ b/gcc/config/riscv/vector.md
> @@ -25,7 +25,6 @@
>  ;; - Intrinsics (https://github.com/riscv/rvv-intrinsic-doc)
>  ;; - Auto-vectorization (autovec.md)
>  ;; - Optimization (autovec-opt.md)
> -;; - VLS patterns (autovec-vls.md)
>
>  (include "vector-iterators.md")
>
> @@ -1210,6 +1209,160 @@
>    [(set_attr "type" "vmov,vlde,vste")
>     (set_attr "mode" "<VT:MODE>")])
>
> +;; -----------------------------------------------------------------
> +;; ---- VLS Moves Operations
> +;; -----------------------------------------------------------------
> +
> +(define_expand "mov<mode>"
> +  [(set (match_operand:VLS_AVL_IMM 0 "reg_or_mem_operand")
> +       (match_operand:VLS_AVL_IMM 1 "general_operand"))]
> +  "TARGET_VECTOR"
> +{
> +  if (riscv_vector::legitimize_move (operands[0], operands[1]))
> +    DONE;
> +})
> +
> +(define_insn_and_split "*mov<mode>_mem_to_mem"
> +  [(set (match_operand:VLS_AVL_IMM 0 "memory_operand")
> +       (match_operand:VLS_AVL_IMM 1 "memory_operand"))]
> +  "TARGET_VECTOR && can_create_pseudo_p ()"
> +  "#"
> +  "&& 1"
> +  [(const_int 0)]
> +  {
> +    if (GET_MODE_BITSIZE (<MODE>mode).to_constant () <= MAX_BITS_PER_WORD)
> +      {
> +        /* Opitmize the following case:
> +
> +           typedef int8_t v2qi __attribute__ ((vector_size (2)));
> +           v2qi v = *(v2qi*)in;
> +           *(v2qi*)out = v;
> +
> +           We prefer scalar load/store instead of vle.v/vse.v when
> +           the VLS modes size is smaller scalar mode.  */
> +        machine_mode mode;
> +        unsigned size = GET_MODE_BITSIZE (<MODE>mode).to_constant ();
> +        if (FLOAT_MODE_P (<MODE>mode))
> +         mode = mode_for_size (size, MODE_FLOAT, 0).require ();
> +        else
> +         mode = mode_for_size (size, MODE_INT, 0).require ();
> +        emit_move_insn (gen_lowpart (mode, operands[0]),
> +                       gen_lowpart (mode, operands[1]));
> +      }
> +    else
> +      {
> +       operands[1] = force_reg (<MODE>mode, operands[1]);
> +       emit_move_insn (operands[0], operands[1]);
> +      }
> +    DONE;
> +  }
> +  [(set_attr "type" "vmov")]
> +)
> +
> +(define_insn_and_split "*mov<mode>"
> +  [(set (match_operand:VLS_AVL_IMM 0 "reg_or_mem_operand" "=vr, m, vr")
> +       (match_operand:VLS_AVL_IMM 1 "reg_or_mem_operand" "  m,vr, vr"))]
> +  "TARGET_VECTOR
> +   && (register_operand (operands[0], <MODE>mode)
> +       || register_operand (operands[1], <MODE>mode))"
> +  "@
> +   #
> +   #
> +   vmv%m1r.v\t%0,%1"
> +  "&& reload_completed
> +   && (!register_operand (operands[0], <MODE>mode)
> +       || !register_operand (operands[1], <MODE>mode))"
> +  [(const_int 0)]
> +  {
> +    bool ok_p = riscv_vector::legitimize_move (operands[0], operands[1]);
> +    gcc_assert (ok_p);
> +    DONE;
> +  }
> +  [(set_attr "type" "vmov")]
> +)
> +
> +(define_expand "mov<mode>"
> +  [(set (match_operand:VLS_AVL_REG 0 "reg_or_mem_operand")
> +       (match_operand:VLS_AVL_REG 1 "general_operand"))]
> +  "TARGET_VECTOR"
> +{
> +  bool ok_p = riscv_vector::legitimize_move (operands[0], operands[1]);
> +  gcc_assert (ok_p);
> +  DONE;
> +})
> +
> +(define_expand "@mov<VLS_AVL_REG:mode><P:mode>_lra"
> +  [(parallel
> +    [(set (match_operand:VLS_AVL_REG 0 "reg_or_mem_operand")
> +         (match_operand:VLS_AVL_REG 1 "reg_or_mem_operand"))
> +   (clobber (match_scratch:P 2))])]
> +  "TARGET_VECTOR && (lra_in_progress || reload_completed)"
> +{})
> +
> +(define_insn_and_split "*mov<VLS_AVL_REG:mode><P:mode>_lra"
> +  [(set (match_operand:VLS_AVL_REG 0 "reg_or_mem_operand" "=vr, m,vr")
> +       (match_operand:VLS_AVL_REG 1 "reg_or_mem_operand" "  m,vr,vr"))
> +   (clobber (match_scratch:P 2 "=&r,&r,X"))]
> +  "TARGET_VECTOR && (lra_in_progress || reload_completed)
> +   && (register_operand (operands[0], <VLS_AVL_REG:MODE>mode)
> +       || register_operand (operands[1], <VLS_AVL_REG:MODE>mode))"
> +  "#"
> +  "&& reload_completed"
> +  [(const_int 0)]
> +{
> +  if (REG_P (operands[0]) && REG_P (operands[1]))
> +      emit_insn (gen_rtx_SET (operands[0], operands[1]));
> +  else
> +    {
> +      emit_move_insn (operands[2], gen_int_mode (GET_MODE_NUNITS (<VLS_AVL_REG:MODE>mode),
> +                                                Pmode));
> +      unsigned insn_flags
> +        = GET_MODE_CLASS (<VLS_AVL_REG:MODE>mode) == MODE_VECTOR_BOOL
> +                                                    ? riscv_vector::UNARY_MASK_OP
> +                                                    : riscv_vector::UNARY_OP;
> +      riscv_vector::emit_nonvlmax_insn (code_for_pred_mov (<VLS_AVL_REG:MODE>mode),
> +                                       insn_flags, operands, operands[2]);
> +    }
> +  DONE;
> +}
> +  [(set_attr "type" "vmov")]
> +)
> +
> +(define_insn "*mov<mode>_vls"
> +  [(set (match_operand:VLS 0 "register_operand" "=vr")
> +       (match_operand:VLS 1 "register_operand" " vr"))]
> +  "TARGET_VECTOR"
> +  "vmv%m1r.v\t%0,%1"
> +  [(set_attr "type" "vmov")
> +   (set_attr "mode" "<MODE>")])
> +
> +(define_insn "*mov<mode>_vls"
> +  [(set (match_operand:VLSB 0 "register_operand" "=vr")
> +       (match_operand:VLSB 1 "register_operand" " vr"))]
> +  "TARGET_VECTOR"
> +  "vmv1r.v\t%0,%1"
> +  [(set_attr "type" "vmov")
> +   (set_attr "mode" "<MODE>")])
> +
> +(define_expand "movmisalign<mode>"
> +  [(set (match_operand:VLS 0 "nonimmediate_operand")
> +       (match_operand:VLS 1 "general_operand"))]
> +  "TARGET_VECTOR"
> +  {
> +    /* To support misalign data movement, we should use
> +       minimum element alignment load/store.  */
> +    unsigned int size = GET_MODE_SIZE (GET_MODE_INNER (<MODE>mode));
> +    poly_int64 nunits = GET_MODE_NUNITS (<MODE>mode) * size;
> +    machine_mode mode = riscv_vector::get_vector_mode (QImode, nunits).require ();
> +    operands[0] = gen_lowpart (mode, operands[0]);
> +    operands[1] = gen_lowpart (mode, operands[1]);
> +    if (MEM_P (operands[0]) && !register_operand (operands[1], mode))
> +      operands[1] = force_reg (mode, operands[1]);
> +    riscv_vector::emit_vlmax_insn (code_for_pred_mov (mode), riscv_vector::UNARY_OP, operands);
> +    DONE;
> +  }
> +)
> +
>  ;; -----------------------------------------------------------------
>  ;; ---- Duplicate Operations
>  ;; -----------------------------------------------------------------
> @@ -1230,6 +1383,22 @@
>    }
>  )
>
> +(define_insn_and_split "@vec_duplicate<mode>"
> +  [(set (match_operand:VLS 0 "register_operand")
> +        (vec_duplicate:VLS
> +          (match_operand:<VEL> 1 "reg_or_int_operand")))]
> +  "TARGET_VECTOR && can_create_pseudo_p ()"
> +  "#"
> +  "&& 1"
> +  [(const_int 0)]
> +  {
> +    riscv_vector::emit_vlmax_insn (code_for_pred_broadcast (<MODE>mode),
> +                                   riscv_vector::UNARY_OP, operands);
> +    DONE;
> +  }
> +  [(set_attr "type" "vector")]
> +)
> +
>  ;; -----------------------------------------------------------------
>  ;; ---- 6. Configuration-Setting Instructions
>  ;; -----------------------------------------------------------------
> @@ -8540,4 +8709,3 @@
>
>  (include "autovec.md")
>  (include "autovec-opt.md")
> -(include "autovec-vls.md")
> --
> 2.36.3
>
  
Li, Pan2 via Gcc-patches Sept. 18, 2023, 8:26 a.m. UTC | #2
Committed, thanks Kito.

Pan


-----Original Message-----
From: Gcc-patches <gcc-patches-bounces+pan2.li=intel.com@gcc.gnu.org> On Behalf Of Kito Cheng via Gcc-patches
Sent: Monday, September 18, 2023 4:01 PM
To: Juzhe-Zhong <juzhe.zhong@rivai.ai>
Cc: gcc-patches@gcc.gnu.org; kito.cheng@gmail.com; jeffreyalaw@gmail.com; rdapp.gcc@gmail.com
Subject: Re: [PATCH] RISC-V: Remove autovec-vls.md file and clean up VLS move modes[NFC]

LGTM :)

On Mon, Sep 18, 2023 at 3:07 PM Juzhe-Zhong <juzhe.zhong@rivai.ai> wrote:
>
> We have largely supportted VLS modes. Only move patterns of VLS modes are
> different from VLS patterns. The rest of them are the same.
>
> We always extend the current VLA patterns with VLSmodes:
>
> VI --> V_VLSI
> VF --> V_VLSF
>
> It makes no sense to have a separate file holding a very few VLS patterns
> that can not be extended from the current VLA patterns.
>
> So remove autovec-vls.md
>
> gcc/ChangeLog:
>
>         * config/riscv/vector.md (mov<mode>): New pattern.
>         (*mov<mode>_mem_to_mem): Ditto.
>         (*mov<mode>): Ditto.
>         (@mov<VLS_AVL_REG:mode><P:mode>_lra): Ditto.
>         (*mov<VLS_AVL_REG:mode><P:mode>_lra): Ditto.
>         (*mov<mode>_vls): Ditto.
>         (movmisalign<mode>): Ditto.
>         (@vec_duplicate<mode>): Ditto.
>         * config/riscv/autovec-vls.md: Removed.
>
> ---
>  gcc/config/riscv/autovec-vls.md | 196 --------------------------------
>  gcc/config/riscv/vector.md      | 172 +++++++++++++++++++++++++++-
>  2 files changed, 170 insertions(+), 198 deletions(-)
>  delete mode 100644 gcc/config/riscv/autovec-vls.md
>
> diff --git a/gcc/config/riscv/autovec-vls.md b/gcc/config/riscv/autovec-vls.md
> deleted file mode 100644
> index 3488f452e5d..00000000000
> --- a/gcc/config/riscv/autovec-vls.md
> +++ /dev/null
> @@ -1,196 +0,0 @@
> -;; Machine description for VLS of RVV auto-vectorization.
> -;; Copyright (C) 2023 Free Software Foundation, Inc.
> -;; Contributed by Juzhe Zhong (juzhe.zhong@rivai.ai), RiVAI Technologies Ltd.
> -
> -;; This file is part of GCC.
> -
> -;; GCC is free software; you can redistribute it and/or modify
> -;; it under the terms of the GNU General Public License as published by
> -;; the Free Software Foundation; either version 3, or (at your option)
> -;; any later version.
> -
> -;; GCC is distributed in the hope that it will be useful,
> -;; but WITHOUT ANY WARRANTY; without even the implied warranty of
> -;; MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> -;; GNU General Public License for more details.
> -
> -;; You should have received a copy of the GNU General Public License
> -;; along with GCC; see the file COPYING3.  If not see
> -;; <http://www.gnu.org/licenses/>.
> -
> -;; We define VLS modes as 'define_insn_and_split' with normal
> -;; RTX_CODE operation, so we can gain benefits from Combine optimizations.
> -
> -;; -----------------------------------------------------------------
> -;; ---- Moves Operations
> -;; -----------------------------------------------------------------
> -
> -(define_expand "mov<mode>"
> -  [(set (match_operand:VLS_AVL_IMM 0 "reg_or_mem_operand")
> -       (match_operand:VLS_AVL_IMM 1 "general_operand"))]
> -  "TARGET_VECTOR"
> -{
> -  if (riscv_vector::legitimize_move (operands[0], operands[1]))
> -    DONE;
> -})
> -
> -(define_insn_and_split "*mov<mode>_mem_to_mem"
> -  [(set (match_operand:VLS_AVL_IMM 0 "memory_operand")
> -       (match_operand:VLS_AVL_IMM 1 "memory_operand"))]
> -  "TARGET_VECTOR && can_create_pseudo_p ()"
> -  "#"
> -  "&& 1"
> -  [(const_int 0)]
> -  {
> -    if (GET_MODE_BITSIZE (<MODE>mode).to_constant () <= MAX_BITS_PER_WORD)
> -      {
> -        /* Opitmize the following case:
> -
> -           typedef int8_t v2qi __attribute__ ((vector_size (2)));
> -           v2qi v = *(v2qi*)in;
> -           *(v2qi*)out = v;
> -
> -           We prefer scalar load/store instead of vle.v/vse.v when
> -           the VLS modes size is smaller scalar mode.  */
> -        machine_mode mode;
> -        unsigned size = GET_MODE_BITSIZE (<MODE>mode).to_constant ();
> -        if (FLOAT_MODE_P (<MODE>mode))
> -         mode = mode_for_size (size, MODE_FLOAT, 0).require ();
> -        else
> -         mode = mode_for_size (size, MODE_INT, 0).require ();
> -        emit_move_insn (gen_lowpart (mode, operands[0]),
> -                       gen_lowpart (mode, operands[1]));
> -      }
> -    else
> -      {
> -       operands[1] = force_reg (<MODE>mode, operands[1]);
> -       emit_move_insn (operands[0], operands[1]);
> -      }
> -    DONE;
> -  }
> -  [(set_attr "type" "vmov")]
> -)
> -
> -(define_insn_and_split "*mov<mode>"
> -  [(set (match_operand:VLS_AVL_IMM 0 "reg_or_mem_operand" "=vr, m, vr")
> -       (match_operand:VLS_AVL_IMM 1 "reg_or_mem_operand" "  m,vr, vr"))]
> -  "TARGET_VECTOR
> -   && (register_operand (operands[0], <MODE>mode)
> -       || register_operand (operands[1], <MODE>mode))"
> -  "@
> -   #
> -   #
> -   vmv%m1r.v\t%0,%1"
> -  "&& reload_completed
> -   && (!register_operand (operands[0], <MODE>mode)
> -       || !register_operand (operands[1], <MODE>mode))"
> -  [(const_int 0)]
> -  {
> -    bool ok_p = riscv_vector::legitimize_move (operands[0], operands[1]);
> -    gcc_assert (ok_p);
> -    DONE;
> -  }
> -  [(set_attr "type" "vmov")]
> -)
> -
> -(define_expand "mov<mode>"
> -  [(set (match_operand:VLS_AVL_REG 0 "reg_or_mem_operand")
> -       (match_operand:VLS_AVL_REG 1 "general_operand"))]
> -  "TARGET_VECTOR"
> -{
> -  bool ok_p = riscv_vector::legitimize_move (operands[0], operands[1]);
> -  gcc_assert (ok_p);
> -  DONE;
> -})
> -
> -(define_expand "@mov<VLS_AVL_REG:mode><P:mode>_lra"
> -  [(parallel
> -    [(set (match_operand:VLS_AVL_REG 0 "reg_or_mem_operand")
> -         (match_operand:VLS_AVL_REG 1 "reg_or_mem_operand"))
> -   (clobber (match_scratch:P 2))])]
> -  "TARGET_VECTOR && (lra_in_progress || reload_completed)"
> -{})
> -
> -(define_insn_and_split "*mov<VLS_AVL_REG:mode><P:mode>_lra"
> -  [(set (match_operand:VLS_AVL_REG 0 "reg_or_mem_operand" "=vr, m,vr")
> -       (match_operand:VLS_AVL_REG 1 "reg_or_mem_operand" "  m,vr,vr"))
> -   (clobber (match_scratch:P 2 "=&r,&r,X"))]
> -  "TARGET_VECTOR && (lra_in_progress || reload_completed)
> -   && (register_operand (operands[0], <VLS_AVL_REG:MODE>mode)
> -       || register_operand (operands[1], <VLS_AVL_REG:MODE>mode))"
> -  "#"
> -  "&& reload_completed"
> -  [(const_int 0)]
> -{
> -  if (REG_P (operands[0]) && REG_P (operands[1]))
> -      emit_insn (gen_rtx_SET (operands[0], operands[1]));
> -  else
> -    {
> -      emit_move_insn (operands[2], gen_int_mode (GET_MODE_NUNITS (<VLS_AVL_REG:MODE>mode),
> -                                                Pmode));
> -      unsigned insn_flags
> -        = GET_MODE_CLASS (<VLS_AVL_REG:MODE>mode) == MODE_VECTOR_BOOL
> -                                                    ? riscv_vector::UNARY_MASK_OP
> -                                                    : riscv_vector::UNARY_OP;
> -      riscv_vector::emit_nonvlmax_insn (code_for_pred_mov (<VLS_AVL_REG:MODE>mode),
> -                                       insn_flags, operands, operands[2]);
> -    }
> -  DONE;
> -}
> -  [(set_attr "type" "vmov")]
> -)
> -
> -(define_insn "*mov<mode>_vls"
> -  [(set (match_operand:VLS 0 "register_operand" "=vr")
> -       (match_operand:VLS 1 "register_operand" " vr"))]
> -  "TARGET_VECTOR"
> -  "vmv%m1r.v\t%0,%1"
> -  [(set_attr "type" "vmov")
> -   (set_attr "mode" "<MODE>")])
> -
> -(define_insn "*mov<mode>_vls"
> -  [(set (match_operand:VLSB 0 "register_operand" "=vr")
> -       (match_operand:VLSB 1 "register_operand" " vr"))]
> -  "TARGET_VECTOR"
> -  "vmv1r.v\t%0,%1"
> -  [(set_attr "type" "vmov")
> -   (set_attr "mode" "<MODE>")])
> -
> -(define_expand "movmisalign<mode>"
> -  [(set (match_operand:VLS 0 "nonimmediate_operand")
> -       (match_operand:VLS 1 "general_operand"))]
> -  "TARGET_VECTOR"
> -  {
> -    /* To support misalign data movement, we should use
> -       minimum element alignment load/store.  */
> -    unsigned int size = GET_MODE_SIZE (GET_MODE_INNER (<MODE>mode));
> -    poly_int64 nunits = GET_MODE_NUNITS (<MODE>mode) * size;
> -    machine_mode mode = riscv_vector::get_vector_mode (QImode, nunits).require ();
> -    operands[0] = gen_lowpart (mode, operands[0]);
> -    operands[1] = gen_lowpart (mode, operands[1]);
> -    if (MEM_P (operands[0]) && !register_operand (operands[1], mode))
> -      operands[1] = force_reg (mode, operands[1]);
> -    riscv_vector::emit_vlmax_insn (code_for_pred_mov (mode), riscv_vector::UNARY_OP, operands);
> -    DONE;
> -  }
> -)
> -
> -;; -----------------------------------------------------------------
> -;; ---- Duplicate Operations
> -;; -----------------------------------------------------------------
> -
> -(define_insn_and_split "@vec_duplicate<mode>"
> -  [(set (match_operand:VLS 0 "register_operand")
> -        (vec_duplicate:VLS
> -          (match_operand:<VEL> 1 "reg_or_int_operand")))]
> -  "TARGET_VECTOR && can_create_pseudo_p ()"
> -  "#"
> -  "&& 1"
> -  [(const_int 0)]
> -  {
> -    riscv_vector::emit_vlmax_insn (code_for_pred_broadcast (<MODE>mode),
> -                                   riscv_vector::UNARY_OP, operands);
> -    DONE;
> -  }
> -  [(set_attr "type" "vector")]
> -)
> diff --git a/gcc/config/riscv/vector.md b/gcc/config/riscv/vector.md
> index 939e992cad1..40512e8d864 100644
> --- a/gcc/config/riscv/vector.md
> +++ b/gcc/config/riscv/vector.md
> @@ -25,7 +25,6 @@
>  ;; - Intrinsics (https://github.com/riscv/rvv-intrinsic-doc)
>  ;; - Auto-vectorization (autovec.md)
>  ;; - Optimization (autovec-opt.md)
> -;; - VLS patterns (autovec-vls.md)
>
>  (include "vector-iterators.md")
>
> @@ -1210,6 +1209,160 @@
>    [(set_attr "type" "vmov,vlde,vste")
>     (set_attr "mode" "<VT:MODE>")])
>
> +;; -----------------------------------------------------------------
> +;; ---- VLS Moves Operations
> +;; -----------------------------------------------------------------
> +
> +(define_expand "mov<mode>"
> +  [(set (match_operand:VLS_AVL_IMM 0 "reg_or_mem_operand")
> +       (match_operand:VLS_AVL_IMM 1 "general_operand"))]
> +  "TARGET_VECTOR"
> +{
> +  if (riscv_vector::legitimize_move (operands[0], operands[1]))
> +    DONE;
> +})
> +
> +(define_insn_and_split "*mov<mode>_mem_to_mem"
> +  [(set (match_operand:VLS_AVL_IMM 0 "memory_operand")
> +       (match_operand:VLS_AVL_IMM 1 "memory_operand"))]
> +  "TARGET_VECTOR && can_create_pseudo_p ()"
> +  "#"
> +  "&& 1"
> +  [(const_int 0)]
> +  {
> +    if (GET_MODE_BITSIZE (<MODE>mode).to_constant () <= MAX_BITS_PER_WORD)
> +      {
> +        /* Opitmize the following case:
> +
> +           typedef int8_t v2qi __attribute__ ((vector_size (2)));
> +           v2qi v = *(v2qi*)in;
> +           *(v2qi*)out = v;
> +
> +           We prefer scalar load/store instead of vle.v/vse.v when
> +           the VLS modes size is smaller scalar mode.  */
> +        machine_mode mode;
> +        unsigned size = GET_MODE_BITSIZE (<MODE>mode).to_constant ();
> +        if (FLOAT_MODE_P (<MODE>mode))
> +         mode = mode_for_size (size, MODE_FLOAT, 0).require ();
> +        else
> +         mode = mode_for_size (size, MODE_INT, 0).require ();
> +        emit_move_insn (gen_lowpart (mode, operands[0]),
> +                       gen_lowpart (mode, operands[1]));
> +      }
> +    else
> +      {
> +       operands[1] = force_reg (<MODE>mode, operands[1]);
> +       emit_move_insn (operands[0], operands[1]);
> +      }
> +    DONE;
> +  }
> +  [(set_attr "type" "vmov")]
> +)
> +
> +(define_insn_and_split "*mov<mode>"
> +  [(set (match_operand:VLS_AVL_IMM 0 "reg_or_mem_operand" "=vr, m, vr")
> +       (match_operand:VLS_AVL_IMM 1 "reg_or_mem_operand" "  m,vr, vr"))]
> +  "TARGET_VECTOR
> +   && (register_operand (operands[0], <MODE>mode)
> +       || register_operand (operands[1], <MODE>mode))"
> +  "@
> +   #
> +   #
> +   vmv%m1r.v\t%0,%1"
> +  "&& reload_completed
> +   && (!register_operand (operands[0], <MODE>mode)
> +       || !register_operand (operands[1], <MODE>mode))"
> +  [(const_int 0)]
> +  {
> +    bool ok_p = riscv_vector::legitimize_move (operands[0], operands[1]);
> +    gcc_assert (ok_p);
> +    DONE;
> +  }
> +  [(set_attr "type" "vmov")]
> +)
> +
> +(define_expand "mov<mode>"
> +  [(set (match_operand:VLS_AVL_REG 0 "reg_or_mem_operand")
> +       (match_operand:VLS_AVL_REG 1 "general_operand"))]
> +  "TARGET_VECTOR"
> +{
> +  bool ok_p = riscv_vector::legitimize_move (operands[0], operands[1]);
> +  gcc_assert (ok_p);
> +  DONE;
> +})
> +
> +(define_expand "@mov<VLS_AVL_REG:mode><P:mode>_lra"
> +  [(parallel
> +    [(set (match_operand:VLS_AVL_REG 0 "reg_or_mem_operand")
> +         (match_operand:VLS_AVL_REG 1 "reg_or_mem_operand"))
> +   (clobber (match_scratch:P 2))])]
> +  "TARGET_VECTOR && (lra_in_progress || reload_completed)"
> +{})
> +
> +(define_insn_and_split "*mov<VLS_AVL_REG:mode><P:mode>_lra"
> +  [(set (match_operand:VLS_AVL_REG 0 "reg_or_mem_operand" "=vr, m,vr")
> +       (match_operand:VLS_AVL_REG 1 "reg_or_mem_operand" "  m,vr,vr"))
> +   (clobber (match_scratch:P 2 "=&r,&r,X"))]
> +  "TARGET_VECTOR && (lra_in_progress || reload_completed)
> +   && (register_operand (operands[0], <VLS_AVL_REG:MODE>mode)
> +       || register_operand (operands[1], <VLS_AVL_REG:MODE>mode))"
> +  "#"
> +  "&& reload_completed"
> +  [(const_int 0)]
> +{
> +  if (REG_P (operands[0]) && REG_P (operands[1]))
> +      emit_insn (gen_rtx_SET (operands[0], operands[1]));
> +  else
> +    {
> +      emit_move_insn (operands[2], gen_int_mode (GET_MODE_NUNITS (<VLS_AVL_REG:MODE>mode),
> +                                                Pmode));
> +      unsigned insn_flags
> +        = GET_MODE_CLASS (<VLS_AVL_REG:MODE>mode) == MODE_VECTOR_BOOL
> +                                                    ? riscv_vector::UNARY_MASK_OP
> +                                                    : riscv_vector::UNARY_OP;
> +      riscv_vector::emit_nonvlmax_insn (code_for_pred_mov (<VLS_AVL_REG:MODE>mode),
> +                                       insn_flags, operands, operands[2]);
> +    }
> +  DONE;
> +}
> +  [(set_attr "type" "vmov")]
> +)
> +
> +(define_insn "*mov<mode>_vls"
> +  [(set (match_operand:VLS 0 "register_operand" "=vr")
> +       (match_operand:VLS 1 "register_operand" " vr"))]
> +  "TARGET_VECTOR"
> +  "vmv%m1r.v\t%0,%1"
> +  [(set_attr "type" "vmov")
> +   (set_attr "mode" "<MODE>")])
> +
> +(define_insn "*mov<mode>_vls"
> +  [(set (match_operand:VLSB 0 "register_operand" "=vr")
> +       (match_operand:VLSB 1 "register_operand" " vr"))]
> +  "TARGET_VECTOR"
> +  "vmv1r.v\t%0,%1"
> +  [(set_attr "type" "vmov")
> +   (set_attr "mode" "<MODE>")])
> +
> +(define_expand "movmisalign<mode>"
> +  [(set (match_operand:VLS 0 "nonimmediate_operand")
> +       (match_operand:VLS 1 "general_operand"))]
> +  "TARGET_VECTOR"
> +  {
> +    /* To support misalign data movement, we should use
> +       minimum element alignment load/store.  */
> +    unsigned int size = GET_MODE_SIZE (GET_MODE_INNER (<MODE>mode));
> +    poly_int64 nunits = GET_MODE_NUNITS (<MODE>mode) * size;
> +    machine_mode mode = riscv_vector::get_vector_mode (QImode, nunits).require ();
> +    operands[0] = gen_lowpart (mode, operands[0]);
> +    operands[1] = gen_lowpart (mode, operands[1]);
> +    if (MEM_P (operands[0]) && !register_operand (operands[1], mode))
> +      operands[1] = force_reg (mode, operands[1]);
> +    riscv_vector::emit_vlmax_insn (code_for_pred_mov (mode), riscv_vector::UNARY_OP, operands);
> +    DONE;
> +  }
> +)
> +
>  ;; -----------------------------------------------------------------
>  ;; ---- Duplicate Operations
>  ;; -----------------------------------------------------------------
> @@ -1230,6 +1383,22 @@
>    }
>  )
>
> +(define_insn_and_split "@vec_duplicate<mode>"
> +  [(set (match_operand:VLS 0 "register_operand")
> +        (vec_duplicate:VLS
> +          (match_operand:<VEL> 1 "reg_or_int_operand")))]
> +  "TARGET_VECTOR && can_create_pseudo_p ()"
> +  "#"
> +  "&& 1"
> +  [(const_int 0)]
> +  {
> +    riscv_vector::emit_vlmax_insn (code_for_pred_broadcast (<MODE>mode),
> +                                   riscv_vector::UNARY_OP, operands);
> +    DONE;
> +  }
> +  [(set_attr "type" "vector")]
> +)
> +
>  ;; -----------------------------------------------------------------
>  ;; ---- 6. Configuration-Setting Instructions
>  ;; -----------------------------------------------------------------
> @@ -8540,4 +8709,3 @@
>
>  (include "autovec.md")
>  (include "autovec-opt.md")
> -(include "autovec-vls.md")
> --
> 2.36.3
>
  

Patch

diff --git a/gcc/config/riscv/autovec-vls.md b/gcc/config/riscv/autovec-vls.md
deleted file mode 100644
index 3488f452e5d..00000000000
--- a/gcc/config/riscv/autovec-vls.md
+++ /dev/null
@@ -1,196 +0,0 @@ 
-;; Machine description for VLS of RVV auto-vectorization.
-;; Copyright (C) 2023 Free Software Foundation, Inc.
-;; Contributed by Juzhe Zhong (juzhe.zhong@rivai.ai), RiVAI Technologies Ltd.
-
-;; This file is part of GCC.
-
-;; GCC is free software; you can redistribute it and/or modify
-;; it under the terms of the GNU General Public License as published by
-;; the Free Software Foundation; either version 3, or (at your option)
-;; any later version.
-
-;; GCC is distributed in the hope that it will be useful,
-;; but WITHOUT ANY WARRANTY; without even the implied warranty of
-;; MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-;; GNU General Public License for more details.
-
-;; You should have received a copy of the GNU General Public License
-;; along with GCC; see the file COPYING3.  If not see
-;; <http://www.gnu.org/licenses/>.
-
-;; We define VLS modes as 'define_insn_and_split' with normal
-;; RTX_CODE operation, so we can gain benefits from Combine optimizations.
-
-;; -----------------------------------------------------------------
-;; ---- Moves Operations
-;; -----------------------------------------------------------------
-
-(define_expand "mov<mode>"
-  [(set (match_operand:VLS_AVL_IMM 0 "reg_or_mem_operand")
-	(match_operand:VLS_AVL_IMM 1 "general_operand"))]
-  "TARGET_VECTOR"
-{
-  if (riscv_vector::legitimize_move (operands[0], operands[1]))
-    DONE;
-})
-
-(define_insn_and_split "*mov<mode>_mem_to_mem"
-  [(set (match_operand:VLS_AVL_IMM 0 "memory_operand")
-	(match_operand:VLS_AVL_IMM 1 "memory_operand"))]
-  "TARGET_VECTOR && can_create_pseudo_p ()"
-  "#"
-  "&& 1"
-  [(const_int 0)]
-  {
-    if (GET_MODE_BITSIZE (<MODE>mode).to_constant () <= MAX_BITS_PER_WORD)
-      {
-        /* Opitmize the following case:
-
-	    typedef int8_t v2qi __attribute__ ((vector_size (2)));
-	    v2qi v = *(v2qi*)in;
-	    *(v2qi*)out = v;
-
-	    We prefer scalar load/store instead of vle.v/vse.v when
-	    the VLS modes size is smaller scalar mode.  */
-        machine_mode mode;
-        unsigned size = GET_MODE_BITSIZE (<MODE>mode).to_constant ();
-        if (FLOAT_MODE_P (<MODE>mode))
-	  mode = mode_for_size (size, MODE_FLOAT, 0).require ();
-        else
-	  mode = mode_for_size (size, MODE_INT, 0).require ();
-        emit_move_insn (gen_lowpart (mode, operands[0]),
-		        gen_lowpart (mode, operands[1]));
-      }
-    else
-      {
-	operands[1] = force_reg (<MODE>mode, operands[1]);
-	emit_move_insn (operands[0], operands[1]);
-      }
-    DONE;
-  }
-  [(set_attr "type" "vmov")]
-)
-
-(define_insn_and_split "*mov<mode>"
-  [(set (match_operand:VLS_AVL_IMM 0 "reg_or_mem_operand" "=vr, m, vr")
-	(match_operand:VLS_AVL_IMM 1 "reg_or_mem_operand" "  m,vr, vr"))]
-  "TARGET_VECTOR
-   && (register_operand (operands[0], <MODE>mode)
-       || register_operand (operands[1], <MODE>mode))"
-  "@
-   #
-   #
-   vmv%m1r.v\t%0,%1"
-  "&& reload_completed
-   && (!register_operand (operands[0], <MODE>mode)
-       || !register_operand (operands[1], <MODE>mode))"
-  [(const_int 0)]
-  {
-    bool ok_p = riscv_vector::legitimize_move (operands[0], operands[1]);
-    gcc_assert (ok_p);
-    DONE;
-  }
-  [(set_attr "type" "vmov")]
-)
-
-(define_expand "mov<mode>"
-  [(set (match_operand:VLS_AVL_REG 0 "reg_or_mem_operand")
-	(match_operand:VLS_AVL_REG 1 "general_operand"))]
-  "TARGET_VECTOR"
-{
-  bool ok_p = riscv_vector::legitimize_move (operands[0], operands[1]);
-  gcc_assert (ok_p);
-  DONE;
-})
-
-(define_expand "@mov<VLS_AVL_REG:mode><P:mode>_lra"
-  [(parallel
-    [(set (match_operand:VLS_AVL_REG 0 "reg_or_mem_operand")
-	  (match_operand:VLS_AVL_REG 1 "reg_or_mem_operand"))
-   (clobber (match_scratch:P 2))])]
-  "TARGET_VECTOR && (lra_in_progress || reload_completed)"
-{})
-
-(define_insn_and_split "*mov<VLS_AVL_REG:mode><P:mode>_lra"
-  [(set (match_operand:VLS_AVL_REG 0 "reg_or_mem_operand" "=vr, m,vr")
-	(match_operand:VLS_AVL_REG 1 "reg_or_mem_operand" "  m,vr,vr"))
-   (clobber (match_scratch:P 2 "=&r,&r,X"))]
-  "TARGET_VECTOR && (lra_in_progress || reload_completed)
-   && (register_operand (operands[0], <VLS_AVL_REG:MODE>mode)
-       || register_operand (operands[1], <VLS_AVL_REG:MODE>mode))"
-  "#"
-  "&& reload_completed"
-  [(const_int 0)]
-{
-  if (REG_P (operands[0]) && REG_P (operands[1]))
-      emit_insn (gen_rtx_SET (operands[0], operands[1]));
-  else
-    {
-      emit_move_insn (operands[2], gen_int_mode (GET_MODE_NUNITS (<VLS_AVL_REG:MODE>mode),
-						 Pmode));
-      unsigned insn_flags
-        = GET_MODE_CLASS (<VLS_AVL_REG:MODE>mode) == MODE_VECTOR_BOOL
-						     ? riscv_vector::UNARY_MASK_OP
-						     : riscv_vector::UNARY_OP;
-      riscv_vector::emit_nonvlmax_insn (code_for_pred_mov (<VLS_AVL_REG:MODE>mode),
-					insn_flags, operands, operands[2]);
-    }
-  DONE;
-}
-  [(set_attr "type" "vmov")]
-)
-
-(define_insn "*mov<mode>_vls"
-  [(set (match_operand:VLS 0 "register_operand" "=vr")
-	(match_operand:VLS 1 "register_operand" " vr"))]
-  "TARGET_VECTOR"
-  "vmv%m1r.v\t%0,%1"
-  [(set_attr "type" "vmov")
-   (set_attr "mode" "<MODE>")])
-
-(define_insn "*mov<mode>_vls"
-  [(set (match_operand:VLSB 0 "register_operand" "=vr")
-	(match_operand:VLSB 1 "register_operand" " vr"))]
-  "TARGET_VECTOR"
-  "vmv1r.v\t%0,%1"
-  [(set_attr "type" "vmov")
-   (set_attr "mode" "<MODE>")])
-
-(define_expand "movmisalign<mode>"
-  [(set (match_operand:VLS 0 "nonimmediate_operand")
-	(match_operand:VLS 1 "general_operand"))]
-  "TARGET_VECTOR"
-  {
-    /* To support misalign data movement, we should use
-       minimum element alignment load/store.  */
-    unsigned int size = GET_MODE_SIZE (GET_MODE_INNER (<MODE>mode));
-    poly_int64 nunits = GET_MODE_NUNITS (<MODE>mode) * size;
-    machine_mode mode = riscv_vector::get_vector_mode (QImode, nunits).require ();
-    operands[0] = gen_lowpart (mode, operands[0]);
-    operands[1] = gen_lowpart (mode, operands[1]);
-    if (MEM_P (operands[0]) && !register_operand (operands[1], mode))
-      operands[1] = force_reg (mode, operands[1]);
-    riscv_vector::emit_vlmax_insn (code_for_pred_mov (mode), riscv_vector::UNARY_OP, operands);
-    DONE;
-  }
-)
-
-;; -----------------------------------------------------------------
-;; ---- Duplicate Operations
-;; -----------------------------------------------------------------
-
-(define_insn_and_split "@vec_duplicate<mode>"
-  [(set (match_operand:VLS 0 "register_operand")
-        (vec_duplicate:VLS
-          (match_operand:<VEL> 1 "reg_or_int_operand")))]
-  "TARGET_VECTOR && can_create_pseudo_p ()"
-  "#"
-  "&& 1"
-  [(const_int 0)]
-  {
-    riscv_vector::emit_vlmax_insn (code_for_pred_broadcast (<MODE>mode),
-                                   riscv_vector::UNARY_OP, operands);
-    DONE;
-  }
-  [(set_attr "type" "vector")]
-)
diff --git a/gcc/config/riscv/vector.md b/gcc/config/riscv/vector.md
index 939e992cad1..40512e8d864 100644
--- a/gcc/config/riscv/vector.md
+++ b/gcc/config/riscv/vector.md
@@ -25,7 +25,6 @@ 
 ;; - Intrinsics (https://github.com/riscv/rvv-intrinsic-doc)
 ;; - Auto-vectorization (autovec.md)
 ;; - Optimization (autovec-opt.md)
-;; - VLS patterns (autovec-vls.md)
 
 (include "vector-iterators.md")
 
@@ -1210,6 +1209,160 @@ 
   [(set_attr "type" "vmov,vlde,vste")
    (set_attr "mode" "<VT:MODE>")])
 
+;; -----------------------------------------------------------------
+;; ---- VLS Moves Operations
+;; -----------------------------------------------------------------
+
+(define_expand "mov<mode>"
+  [(set (match_operand:VLS_AVL_IMM 0 "reg_or_mem_operand")
+	(match_operand:VLS_AVL_IMM 1 "general_operand"))]
+  "TARGET_VECTOR"
+{
+  if (riscv_vector::legitimize_move (operands[0], operands[1]))
+    DONE;
+})
+
+(define_insn_and_split "*mov<mode>_mem_to_mem"
+  [(set (match_operand:VLS_AVL_IMM 0 "memory_operand")
+	(match_operand:VLS_AVL_IMM 1 "memory_operand"))]
+  "TARGET_VECTOR && can_create_pseudo_p ()"
+  "#"
+  "&& 1"
+  [(const_int 0)]
+  {
+    if (GET_MODE_BITSIZE (<MODE>mode).to_constant () <= MAX_BITS_PER_WORD)
+      {
+        /* Opitmize the following case:
+
+	    typedef int8_t v2qi __attribute__ ((vector_size (2)));
+	    v2qi v = *(v2qi*)in;
+	    *(v2qi*)out = v;
+
+	    We prefer scalar load/store instead of vle.v/vse.v when
+	    the VLS modes size is smaller scalar mode.  */
+        machine_mode mode;
+        unsigned size = GET_MODE_BITSIZE (<MODE>mode).to_constant ();
+        if (FLOAT_MODE_P (<MODE>mode))
+	  mode = mode_for_size (size, MODE_FLOAT, 0).require ();
+        else
+	  mode = mode_for_size (size, MODE_INT, 0).require ();
+        emit_move_insn (gen_lowpart (mode, operands[0]),
+		        gen_lowpart (mode, operands[1]));
+      }
+    else
+      {
+	operands[1] = force_reg (<MODE>mode, operands[1]);
+	emit_move_insn (operands[0], operands[1]);
+      }
+    DONE;
+  }
+  [(set_attr "type" "vmov")]
+)
+
+(define_insn_and_split "*mov<mode>"
+  [(set (match_operand:VLS_AVL_IMM 0 "reg_or_mem_operand" "=vr, m, vr")
+	(match_operand:VLS_AVL_IMM 1 "reg_or_mem_operand" "  m,vr, vr"))]
+  "TARGET_VECTOR
+   && (register_operand (operands[0], <MODE>mode)
+       || register_operand (operands[1], <MODE>mode))"
+  "@
+   #
+   #
+   vmv%m1r.v\t%0,%1"
+  "&& reload_completed
+   && (!register_operand (operands[0], <MODE>mode)
+       || !register_operand (operands[1], <MODE>mode))"
+  [(const_int 0)]
+  {
+    bool ok_p = riscv_vector::legitimize_move (operands[0], operands[1]);
+    gcc_assert (ok_p);
+    DONE;
+  }
+  [(set_attr "type" "vmov")]
+)
+
+(define_expand "mov<mode>"
+  [(set (match_operand:VLS_AVL_REG 0 "reg_or_mem_operand")
+	(match_operand:VLS_AVL_REG 1 "general_operand"))]
+  "TARGET_VECTOR"
+{
+  bool ok_p = riscv_vector::legitimize_move (operands[0], operands[1]);
+  gcc_assert (ok_p);
+  DONE;
+})
+
+(define_expand "@mov<VLS_AVL_REG:mode><P:mode>_lra"
+  [(parallel
+    [(set (match_operand:VLS_AVL_REG 0 "reg_or_mem_operand")
+	  (match_operand:VLS_AVL_REG 1 "reg_or_mem_operand"))
+   (clobber (match_scratch:P 2))])]
+  "TARGET_VECTOR && (lra_in_progress || reload_completed)"
+{})
+
+(define_insn_and_split "*mov<VLS_AVL_REG:mode><P:mode>_lra"
+  [(set (match_operand:VLS_AVL_REG 0 "reg_or_mem_operand" "=vr, m,vr")
+	(match_operand:VLS_AVL_REG 1 "reg_or_mem_operand" "  m,vr,vr"))
+   (clobber (match_scratch:P 2 "=&r,&r,X"))]
+  "TARGET_VECTOR && (lra_in_progress || reload_completed)
+   && (register_operand (operands[0], <VLS_AVL_REG:MODE>mode)
+       || register_operand (operands[1], <VLS_AVL_REG:MODE>mode))"
+  "#"
+  "&& reload_completed"
+  [(const_int 0)]
+{
+  if (REG_P (operands[0]) && REG_P (operands[1]))
+      emit_insn (gen_rtx_SET (operands[0], operands[1]));
+  else
+    {
+      emit_move_insn (operands[2], gen_int_mode (GET_MODE_NUNITS (<VLS_AVL_REG:MODE>mode),
+						 Pmode));
+      unsigned insn_flags
+        = GET_MODE_CLASS (<VLS_AVL_REG:MODE>mode) == MODE_VECTOR_BOOL
+						     ? riscv_vector::UNARY_MASK_OP
+						     : riscv_vector::UNARY_OP;
+      riscv_vector::emit_nonvlmax_insn (code_for_pred_mov (<VLS_AVL_REG:MODE>mode),
+					insn_flags, operands, operands[2]);
+    }
+  DONE;
+}
+  [(set_attr "type" "vmov")]
+)
+
+(define_insn "*mov<mode>_vls"
+  [(set (match_operand:VLS 0 "register_operand" "=vr")
+	(match_operand:VLS 1 "register_operand" " vr"))]
+  "TARGET_VECTOR"
+  "vmv%m1r.v\t%0,%1"
+  [(set_attr "type" "vmov")
+   (set_attr "mode" "<MODE>")])
+
+(define_insn "*mov<mode>_vls"
+  [(set (match_operand:VLSB 0 "register_operand" "=vr")
+	(match_operand:VLSB 1 "register_operand" " vr"))]
+  "TARGET_VECTOR"
+  "vmv1r.v\t%0,%1"
+  [(set_attr "type" "vmov")
+   (set_attr "mode" "<MODE>")])
+
+(define_expand "movmisalign<mode>"
+  [(set (match_operand:VLS 0 "nonimmediate_operand")
+	(match_operand:VLS 1 "general_operand"))]
+  "TARGET_VECTOR"
+  {
+    /* To support misalign data movement, we should use
+       minimum element alignment load/store.  */
+    unsigned int size = GET_MODE_SIZE (GET_MODE_INNER (<MODE>mode));
+    poly_int64 nunits = GET_MODE_NUNITS (<MODE>mode) * size;
+    machine_mode mode = riscv_vector::get_vector_mode (QImode, nunits).require ();
+    operands[0] = gen_lowpart (mode, operands[0]);
+    operands[1] = gen_lowpart (mode, operands[1]);
+    if (MEM_P (operands[0]) && !register_operand (operands[1], mode))
+      operands[1] = force_reg (mode, operands[1]);
+    riscv_vector::emit_vlmax_insn (code_for_pred_mov (mode), riscv_vector::UNARY_OP, operands);
+    DONE;
+  }
+)
+
 ;; -----------------------------------------------------------------
 ;; ---- Duplicate Operations
 ;; -----------------------------------------------------------------
@@ -1230,6 +1383,22 @@ 
   }
 )
 
+(define_insn_and_split "@vec_duplicate<mode>"
+  [(set (match_operand:VLS 0 "register_operand")
+        (vec_duplicate:VLS
+          (match_operand:<VEL> 1 "reg_or_int_operand")))]
+  "TARGET_VECTOR && can_create_pseudo_p ()"
+  "#"
+  "&& 1"
+  [(const_int 0)]
+  {
+    riscv_vector::emit_vlmax_insn (code_for_pred_broadcast (<MODE>mode),
+                                   riscv_vector::UNARY_OP, operands);
+    DONE;
+  }
+  [(set_attr "type" "vector")]
+)
+
 ;; -----------------------------------------------------------------
 ;; ---- 6. Configuration-Setting Instructions
 ;; -----------------------------------------------------------------
@@ -8540,4 +8709,3 @@ 
 
 (include "autovec.md")
 (include "autovec-opt.md")
-(include "autovec-vls.md")