LGTM

On Sun, Nov 19, 2023 at 1:36 PM Maciej W. Rozycki <ma...@embecosm.com> wrote:
>
> Our `mov<mode>cc' expander is no longer specific to short forward branch
> targets, so move its associated comment accordingly.
>
>         gcc/
>         * config/riscv/riscv.md (mov<mode>cc): Move comment on SFB
>         patterns over to...
>         (*mov<GPR:mode><X:mode>cc): ... here.
> ---
>  gcc/config/riscv/riscv.md |    4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> gcc-riscv-sfb-comment-move.diff
> Index: gcc-master/gcc/config/riscv/riscv.md
> ===================================================================
> --- gcc-master.orig/gcc/config/riscv/riscv.md
> +++ gcc-master/gcc/config/riscv/riscv.md
> @@ -2655,8 +2655,6 @@
>    [(set_attr "type" "branch")
>     (set_attr "mode" "none")])
>
> -;; Patterns for implementations that optimize short forward branches.
> -
>  (define_expand "mov<mode>cc"
>    [(set (match_operand:GPR 0 "register_operand")
>         (if_then_else:GPR (match_operand 1 "comparison_operator")
> @@ -2671,6 +2669,8 @@
>      FAIL;
>  })
>
> +;; Patterns for implementations that optimize short forward branches.
> +
>  (define_insn "*mov<GPR:mode><X:mode>cc"
>    [(set (match_operand:GPR 0 "register_operand" "=r,r")
>         (if_then_else:GPR

Reply via email to