diff mbox series

[3/4] VAX: Use a mode with `const_double_zero' expressions

Message ID alpine.LFD.2.21.2101080134480.1637534@eddie.linux-mips.org
State Accepted
Headers show
Series Fixes for `const_double_zero' use + VAX iterator simplification | expand

Commit Message

Maciej W. Rozycki Jan. 8, 2021, 1:50 a.m. UTC 
For predictable semantics propagate the mode from operands referred by 
the FP substitution to the `const_double_zero' expressions used with the 
associated condition code calculation.  Use an iterator to make copies 
of the FP substitution across the FP modes supported as the substitution 
now has to match the mode of the operands.

	gcc/
	* config/pdp11/pdp11.md (subst_f<cc>): Add mode to operands and 
	`const_double_zero'.
---
 gcc/config/vax/vax.md |    6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

gcc-vax-const-double-zero-mode.diff

Comments

Jeff Law Jan. 8, 2021, 8:11 p.m. UTC | #1 
On 1/7/21 6:50 PM, Maciej W. Rozycki wrote:
> For predictable semantics propagate the mode from operands referred by 
> the FP substitution to the `const_double_zero' expressions used with the 
> associated condition code calculation.  Use an iterator to make copies 
> of the FP substitution across the FP modes supported as the substitution 
> now has to match the mode of the operands.
>
> 	gcc/
> 	* config/pdp11/pdp11.md (subst_f<cc>): Add mode to operands and 
> 	`const_double_zero'.
OK
jeff
diff mbox series

Patch

Index: gcc/gcc/config/vax/vax.md
===================================================================
--- gcc.orig/gcc/config/vax/vax.md
+++ gcc/gcc/config/vax/vax.md
@@ -79,13 +79,13 @@ 
 	(match_dup 1))])
 
 (define_subst "subst_f<cc>"
-  [(set (match_operand 0 "")
-	(match_operand 1 ""))
+  [(set (match_operand:VAXfp 0 "")
+	(match_operand:VAXfp 1 ""))
    (clobber (reg:CC VAX_PSL_REGNUM))]
   ""
   [(set (reg:VAXccnz VAX_PSL_REGNUM)
 	(compare:VAXccnz (match_dup 1)
-			 (const_double_zero)))
+			 (const_double_zero:VAXfp)))
    (set (match_dup 0)
 	(match_dup 1))])