[PATCH 04/11] cris: Update unexpected empty split condition
Hans-Peter Nilsson
hp@axis.com
Wed Jun 2 12:45:46 GMT 2021
> From: Kewen Lin <linkw@linux.ibm.com>
> Date: Wed, 2 Jun 2021 07:04:54 +0200
> gcc/ChangeLog:
>
> * config/cris/cris.md (*addi_reload): Fix empty split condition.
> ---
> gcc/config/cris/cris.md | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/gcc/config/cris/cris.md b/gcc/config/cris/cris.md
> index 7de0ec63fcf..d5a3c703a83 100644
> --- a/gcc/config/cris/cris.md
> +++ b/gcc/config/cris/cris.md
> @@ -1311,7 +1311,7 @@ (define_insn_and_split "*addi_reload"
> && (INTVAL (operands[3]) == 2 || INTVAL (operands[3]) == 4)
> && (reload_in_progress || reload_completed)"
> "#"
> - ""
> + "&& 1"
> [(set (match_dup 0)
> (plus:SI (ashift:SI (match_dup 2) (match_dup 3)) (match_dup 1)))]
> "operands[3] = operands[3] == const2_rtx ? const1_rtx : const2_rtx;")
> --
> 2.17.1
>
Ok, thanks, if only for all-round consistency.
In preparation for a warning for an empty condition? I'm
usually all for .md-warnings, but I'm not sure about the
benefit of that one, though. Those "&& 1" look...hackish.
brgds, H-P
More information about the Gcc-patches
mailing list