Hi,

On 09/06/2016 23:38, Jason Merrill wrote:
I would think that when we see a duplicated base we should just drop
the duplicates and continue.

If the type of b1 is error_mark_node, why isn't the type of b2 also
error_mark_node?
Thanks Jason. Normally check_initializer massages the decl basing on the init and so to speak transforms an error_mark_node as TREE_TYPE of the init into an error_mark_node as DECL_INITIAL of a reference. Thus we don't see error_mark_node as TREE_TYPE of b2. Now if I do something as trivial as the attached the testcase passes.

Next step, analyze the huge breakage caused by that change, I don't dare running the testsuite ;)

Thanks,
Paolo.

///////////////////


Index: decl.c
===================================================================
--- decl.c      (revision 237280)
+++ decl.c      (working copy)
@@ -6002,6 +6002,12 @@ check_initializer (tree decl, tree init, int flags
   tree init_code = NULL;
   tree core_type;
 
+  if (TREE_TYPE (init) == error_mark_node)
+    {
+      TREE_TYPE (decl) = error_mark_node;
+      return NULL_TREE;
+    }
+
   /* Things that are going to be initialized need to have complete
      type.  */
   TREE_TYPE (decl) = type = complete_type (TREE_TYPE (decl));

Reply via email to