Re: [PATCH 1/2] xtensa: Simplify several MD templates

[Max Filippov <jcmvbkbc@gmail.com>]

On Fri, May 31, 2024 at 07:23:13PM +0900, Takayuki 'January June' Suwa wrote:
> No functional changes.
> 
> gcc/ChangeLog:
> 
> 	* config/xtensa/predicates.md
> 	(subreg_HQI_lowpart_operator, xtensa_sminmax_operator):
> 	New operator predicates.
> 	* config/xtensa/xtensa-protos.h (xtensa_match_CLAMPS_imms_p):
> 	Remove.
> 	* config/xtensa/xtensa.cc (xtensa_match_CLAMPS_imms_p): Ditto.
> 	* config/xtensa/xtensa.md
> 	(*addsubx, *extzvsi-1bit_ashlsi3, *extzvsi-1bit_addsubx):
> 	Revise the output statements by conditional ternary operator rather
> 	than switch-case clause in order to avoid using gcc_unreachable().
> 	(xtensa_clamps): Reduce to a single pattern definition using the
> 	predicate added above.
> 	(Some split patterns to assist *masktrue_const_bitcmpl): Ditto.
> ---
>  gcc/config/xtensa/predicates.md   |  23 +++++++
>  gcc/config/xtensa/xtensa-protos.h |   1 -
>  gcc/config/xtensa/xtensa.cc       |  10 ---
>  gcc/config/xtensa/xtensa.md       | 109 ++++++------------------------
>  4 files changed, 43 insertions(+), 100 deletions(-)

Regtested for target=xtensa-linux-uclibc, no new regressions.
Committed to master.

Suwa-san, something has changed in your mail setup apparently: every
patch context line now gets an extra space in the beginning. Could you
please fix that?

-- 
Thanks.
-- Max