https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88025

kargl at gcc dot gnu.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|P3                          |P4
             Status|UNCONFIRMED                 |NEW
   Last reconfirmed|                            |2018-11-14
     Ever confirmed|0                           |1

--- Comment #1 from kargl at gcc dot gnu.org ---
The 2 of the 3 asserts aren't needed.  The third is silly.

Index: gcc/fortran/expr.c
===================================================================
--- gcc/fortran/expr.c  (revision 266155)
+++ gcc/fortran/expr.c  (working copy)
@@ -4461,12 +4461,10 @@ gfc_apply_init (gfc_typespec *ts, symbol_attribute *at
 {
   if (ts->type == BT_CHARACTER && !attr->pointer && init
       && ts->u.cl
-      && ts->u.cl->length && ts->u.cl->length->expr_type == EXPR_CONSTANT)
+      && ts->u.cl->length
+      && ts->u.cl->length->expr_type == EXPR_CONSTANT
+      && ts->u.cl->length->ts.type == BT_INTEGER)
     {
-      gcc_assert (ts->u.cl && ts->u.cl->length);
-      gcc_assert (ts->u.cl->length->expr_type == EXPR_CONSTANT);
-      gcc_assert (ts->u.cl->length->ts.type == BT_INTEGER);
-
       HOST_WIDE_INT len = gfc_mpz_get_hwi (ts->u.cl->length->value.integer);

       if (init->expr_type == EXPR_CONSTANT)

Reply via email to