diff --git a/gcc/fortran/ChangeLog b/gcc/fortran/ChangeLog
index ea1afe18cef1ad0d0487ac9ead58b422bcc7bcc0..cc6824bb8b217e09d3514e0c21b688cc468d31d9 100644
--- a/gcc/fortran/ChangeLog
+++ b/gcc/fortran/ChangeLog
@@ -1,3 +1,8 @@
+2005-12-13  Richard Guenther  <rguenther@suse.de>
+
+	* trans-expr.c (gfc_conv_substring): Use fold_build2 and
+	build_int_cst.
+
 2005-12-13  Richard Sandiford  <richard@codesourcery.com>
 
 	* Make-lang.in (fortran/trans-resolve.o): Depend on
diff --git a/gcc/fortran/trans-expr.c b/gcc/fortran/trans-expr.c
index 5e1535e7ea369c1fd5df3b750bb24f3d85424e9e..7def3ad6e805f68920f8ada0543660c9e6478fcc 100644
--- a/gcc/fortran/trans-expr.c
+++ b/gcc/fortran/trans-expr.c
@@ -246,12 +246,11 @@ gfc_conv_substring (gfc_se * se, gfc_ref * ref, int kind)
       gfc_conv_expr_type (&end, ref->u.ss.end, gfc_charlen_type_node);
       gfc_add_block_to_block (&se->pre, &end.pre);
     }
-  tmp =
-    build2 (MINUS_EXPR, gfc_charlen_type_node,
-	    fold_convert (gfc_charlen_type_node, integer_one_node),
-	    start.expr);
-  tmp = build2 (PLUS_EXPR, gfc_charlen_type_node, end.expr, tmp);
-  se->string_length = fold (tmp);
+  tmp = fold_build2 (MINUS_EXPR, gfc_charlen_type_node,
+		     build_int_cst (gfc_charlen_type_node, 1),
+		     start.expr);
+  tmp = fold_build2 (PLUS_EXPR, gfc_charlen_type_node, end.expr, tmp);
+  se->string_length = tmp;
 }