Re: [PATCH 03/11] riscv: xtheadmempair: Fix doc for th_mempair_order_operands()

[Kito Cheng <kito.cheng@gmail.com>]

LGTM

On Fri, Apr 28, 2023 at 2:17 PM Christoph Muellner
<christoph.muellner@vrull.eu> wrote:
>
> From: Christoph Müllner <christoph.muellner@vrull.eu>
>
> There is an incorrect sentence in the documentation of the function
> th_mempair_order_operands(). Let's remove it.
>
> gcc/ChangeLog:
>
>         * config/riscv/thead.cc (th_mempair_operands_p):
>         Fix documentation of th_mempair_order_operands().
>
> Signed-off-by: Christoph Müllner <christoph.muellner@vrull.eu>
> ---
>  gcc/config/riscv/thead.cc | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/gcc/config/riscv/thead.cc b/gcc/config/riscv/thead.cc
> index d7e3cf80d9b..507c912bc39 100644
> --- a/gcc/config/riscv/thead.cc
> +++ b/gcc/config/riscv/thead.cc
> @@ -336,8 +336,8 @@ th_mempair_operands_p (rtx operands[4], bool load_p,
>  }
>
>  /* Given OPERANDS of consecutive load/store that can be merged,
> -   swap them if they are not in ascending order.
> -   Return true if swap was performed.  */
> +   swap them if they are not in ascending order.  */
> +
>  void
>  th_mempair_order_operands (rtx operands[4], bool load_p, machine_mode mode)
>  {
> --
> 2.40.1
>