Revert "Replace error_at with assert in build_va_arg"
authorTom de Vries <tom@codesourcery.com>
Sun, 4 Sep 2016 09:07:52 +0000 (09:07 +0000)
committerTom de Vries <vries@gcc.gnu.org>
Sun, 4 Sep 2016 09:07:52 +0000 (09:07 +0000)
2016-09-04  Tom de Vries  <tom@codesourcery.com>

revert:
2016-08-29  Tom de Vries  <tom@codesourcery.com>

* c-common.c (build_va_arg): Replace first argument type error
with assert.

From-SVN: r239975

gcc/c-family/ChangeLog
gcc/c-family/c-common.c

index 9dc1fdb55440d4cafa9376d3224393d1e358aaac..7a641403cdadb94e3977bdefc710a1ee5163712a 100644 (file)
@@ -1,3 +1,11 @@
+2016-09-04  Tom de Vries  <tom@codesourcery.com>
+
+       revert:
+       2016-08-29  Tom de Vries  <tom@codesourcery.com>
+
+       * c-common.c (build_va_arg): Replace first argument type error
+       with assert.
+
 2016-09-02  Jakub Jelinek  <jakub@redhat.com>
 
        PR c/65467
index 399ba975184e934bc061113286787840a7531291..1a834a3735b445d4a38b0c6d6483b336ee02f920 100644 (file)
@@ -5834,7 +5834,12 @@ build_va_arg (location_t loc, tree expr, tree type)
       /* Verify that &ap is still recognized as having va_list type.  */
       tree canon_expr_type
        = targetm.canonical_va_list_type (TREE_TYPE (expr));
-      gcc_assert (canon_expr_type != NULL_TREE);
+      if (canon_expr_type == NULL_TREE)
+       {
+         error_at (loc,
+                   "first argument to %<va_arg%> not of type %<va_list%>");
+         return error_mark_node;
+       }
 
       return build_va_arg_1 (loc, type, expr);
     }
@@ -5902,7 +5907,12 @@ build_va_arg (location_t loc, tree expr, tree type)
       /* Verify that &ap is still recognized as having va_list type.  */
       tree canon_expr_type
        = targetm.canonical_va_list_type (TREE_TYPE (expr));
-      gcc_assert (canon_expr_type != NULL_TREE);
+      if (canon_expr_type == NULL_TREE)
+       {
+         error_at (loc,
+                   "first argument to %<va_arg%> not of type %<va_list%>");
+         return error_mark_node;
+       }
     }
   else
     {