diff options
| author | Richard Kenner <kenner@gcc.gnu.org> | 1995-01-10 05:49:13 -0500 |
|---|---|---|
| committer | Richard Kenner <kenner@gcc.gnu.org> | 1995-01-10 05:49:13 -0500 |
| commit | 7f0b305c5a35094bc999835d1ab1fa942e6d7ebf (patch) | |
| tree | e96a08980bab3483aba8b666e5a050992badd375 /gcc/fold-const.c | |
| parent | dddd8006abd2dc2a30a9a8323c2363803716b21f (diff) | |
| download | gcc-7f0b305c5a35094bc999835d1ab1fa942e6d7ebf.zip gcc-7f0b305c5a35094bc999835d1ab1fa942e6d7ebf.tar.gz gcc-7f0b305c5a35094bc999835d1ab1fa942e6d7ebf.tar.bz2 | |
(fold): Avoid infinite recursion caused by last change.
From-SVN: r8730
Diffstat (limited to 'gcc/fold-const.c')
| -rw-r--r-- | gcc/fold-const.c | 12 |
1 files changed, 7 insertions, 5 deletions
diff --git a/gcc/fold-const.c b/gcc/fold-const.c index 6b12c42..4272336 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: |