procyberian/gcc
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

xtensa: The use of CLAMPS instruction also requires TARGET_MINMAX, as well as TARGET_CLAMPS

Browse source 
Because both smin and smax requiring TARGET_MINMAX are essential to the
RTL representation.

gcc/ChangeLog:

	* config/xtensa/xtensa.cc (xtensa_match_CLAMPS_imms_p):
	Simplify.
	* config/xtensa/xtensa.md (*xtensa_clamps):
	Add TARGET_MINMAX to the condition.
This commit is contained in:
Takayuki 'January June' Suwa 2023-07-02 02:20:08 +09:00 • committed by Max Filippov
parent 0778f4dcda
commit 2f615b33dd
2 changed files with 4 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
gcc/config/xtensa/xtensa.cc
View file

@ -2649,11 +2649,8 @@ xtensa_emit_add_imm (rtx dst, rtx src, HOST_WIDE_INT imm, rtx scratch,
bool
xtensa_match_CLAMPS_imms_p (rtx cst_max, rtx cst_min)
{
int max, min;
return IN_RANGE (max = exact_log2 (-INTVAL (cst_max)), 7, 22)
&& IN_RANGE (min = exact_log2 (INTVAL (cst_min) + 1), 7, 22)
&& max == min;
return IN_RANGE (exact_log2 (-INTVAL (cst_max)), 7, 22)
&& (INTVAL (cst_max) + INTVAL (cst_min)) == -1;
}

4
gcc/config/xtensa/xtensa.md
View file

@ -522,7 +522,7 @@
(smax:SI (smin:SI (match_operand:SI 1 "register_operand" "r")
(match_operand:SI 2 "const_int_operand" "i"))
(match_operand:SI 3 "const_int_operand" "i")))]
"TARGET_CLAMPS
"TARGET_MINMAX && TARGET_CLAMPS
&& xtensa_match_CLAMPS_imms_p (operands[3], operands[2])"
"#"
"&& 1"
@ -540,7 +540,7 @@
(smin:SI (smax:SI (match_operand:SI 1 "register_operand" "r")
(match_operand:SI 2 "const_int_operand" "i"))
(match_operand:SI 3 "const_int_operand" "i")))]
"TARGET_CLAMPS
"TARGET_MINMAX && TARGET_CLAMPS
&& xtensa_match_CLAMPS_imms_p (operands[2], operands[3])"
{
static char result[64];