From: Roger Sayle <roger@eyesopen.com>
Date: Wed, 4 Jun 2003 04:40:26 +0000 (+0000)
Subject: optabs.c (expand_binop): Attempt to reuse pseudos for duplicate non-volatile operands... 
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=894207cf0ac9f0a841565779d78777cee2464e55;p=gcc.git

optabs.c (expand_binop): Attempt to reuse pseudos for duplicate non-volatile operands of binary operations.


	* optabs.c (expand_binop): Attempt to reuse pseudos for duplicate
	non-volatile operands of binary operations.
	(prepare_cmp_insn): Likewise.

From-SVN: r67417
---

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 9b20061921e..034d6cc7767 100644
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,9 @@
+2003-06-03  Roger Sayle  <roger@eyesopen.com>
+
+	* optabs.c (expand_binop): Attempt to reuse pseudos for duplicate
+	non-volatile operands of binary operations.
+	(prepare_cmp_insn): Likewise.
+
 2003-06-03  Roger Sayle  <roger@eyesopen.com>
 
 	* varasm.c (force_const_mem): Handle alignment of constants not
diff --git a/gcc/optabs.c b/gcc/optabs.c
index b59f89ca8c0..171d13f2d6e 100644
--- a/gcc/optabs.c
+++ b/gcc/optabs.c
@@ -700,8 +700,17 @@ expand_binop (mode, binoptab, op0, op1, target, unsignedp, methods)
 
   if (flag_force_mem)
     {
-      op0 = force_not_mem (op0);
-      op1 = force_not_mem (op1);
+      /* Load duplicate non-volatile operands once.  */
+      if (rtx_equal_p (op0, op1) && ! volatile_refs_p (op0))
+	{
+	  op0 = force_not_mem (op0);
+	  op1 = op0;
+	}
+      else
+	{
+	  op0 = force_not_mem (op0);
+	  op1 = force_not_mem (op1);
+	}
     }
 
   /* If subtracting an integer constant, convert this into an addition of
@@ -3587,8 +3596,17 @@ prepare_cmp_insn (px, py, pcomparison, size, pmode, punsignedp, purpose)
 
   if (mode != BLKmode && flag_force_mem)
     {
-      x = force_not_mem (x);
-      y = force_not_mem (y);
+      /* Load duplicate non-volatile operands once.  */
+      if (rtx_equal_p (x, y) && ! volatile_refs_p (x))
+	{
+	  x = force_not_mem (x);
+	  y = x;
+	}
+      else
+	{
+	  x = force_not_mem (x);
+	  y = force_not_mem (y);
+	}
     }
 
   /* If we are inside an appropriately-short loop and one operand is an