From: Richard Kenner <kenner@gcc.gnu.org>
Date: Tue, 10 Jan 1995 10:49:13 +0000 (-0500)
Subject: (fold): Avoid infinite recursion caused by last change.
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=7f0b305c5a35094bc999835d1ab1fa942e6d7ebf;p=gcc.git

(fold): Avoid infinite recursion caused by last change.

From-SVN: r8730
---

diff --git a/gcc/fold-const.c b/gcc/fold-const.c
index 6b12c42e24b..42723367389 100644
--- a/gcc/fold-const.c
+++ b/gcc/fold-const.c
@@ -3366,17 +3366,17 @@ fold (expr)
 	  tree inter_type = TREE_TYPE (TREE_OPERAND (t, 0));
 	  tree final_type = TREE_TYPE (t);
 	  int inside_int = INTEGRAL_TYPE_P (inside_type);
-	  int inside_ptr = TREE_CODE (inside_type) == POINTER_TYPE;
+	  int inside_ptr = POINTER_TYPE_P (inside_type);
 	  int inside_float = FLOAT_TYPE_P (inside_type);
 	  int inside_prec = TYPE_PRECISION (inside_type);
 	  int inside_unsignedp = TREE_UNSIGNED (inside_type);
 	  int inter_int = INTEGRAL_TYPE_P (inter_type);
-	  int inter_ptr = TREE_CODE (inter_type) == POINTER_TYPE;
+	  int inter_ptr = POINTER_TYPE_P (inter_type);
 	  int inter_float = FLOAT_TYPE_P (inter_type);
 	  int inter_prec = TYPE_PRECISION (inter_type);
 	  int inter_unsignedp = TREE_UNSIGNED (inter_type);
 	  int final_int = INTEGRAL_TYPE_P (final_type);
-	  int final_ptr = TREE_CODE (final_type) == POINTER_TYPE;
+	  int final_ptr = POINTER_TYPE_P (final_type);
 	  int final_float = FLOAT_TYPE_P (final_type);
 	  int final_prec = TYPE_PRECISION (final_type);
 	  int final_unsignedp = TREE_UNSIGNED (final_type);
@@ -3393,11 +3393,13 @@ fold (expr)
 	  /* Likewise, if the intermediate and final types are either both
 	     float or both integer, we don't need the middle conversion if
 	     it is wider than the final type and doesn't change the signedness
-	     (for integers).  */
+	     (for integers).  Avoid this if the final type is a pointer
+	     since then we sometimes need the inner conversion.  */
 	  if ((((inter_int || inter_ptr) && (inside_int || inside_ptr))
 	       || (inter_float && inside_float))
 	      && inter_prec >= inside_prec
-	      && (inter_float || inter_unsignedp == inside_unsignedp))
+	      && (inter_float || inter_unsignedp == inside_unsignedp)
+	      && ! final_ptr)
 	    return convert (final_type, TREE_OPERAND (TREE_OPERAND (t, 0), 0));
 
 	  /* Two conversions in a row are not needed unless: