PR c++/83937 - wrong C++17 handling of init-list ctor argument.
authorJason Merrill <jason@redhat.com>
Fri, 16 Mar 2018 14:35:47 +0000 (10:35 -0400)
committerJason Merrill <jason@gcc.gnu.org>
Fri, 16 Mar 2018 14:35:47 +0000 (10:35 -0400)
* call.c (build_special_member_call): Don't convert an init-list
argument directly to the class type.

From-SVN: r258594

gcc/cp/ChangeLog
gcc/cp/call.c
gcc/testsuite/g++.dg/cpp0x/initlist-ctor2.C [new file with mode: 0644]

index 0ea83713701f8a2baea0dcef58c9b2f043987dfb..7428c4bf20bc3bb4c84810d2b5e46b0c2a4a3de7 100644 (file)
@@ -1,3 +1,9 @@
+2018-03-16  Jason Merrill  <jason@redhat.com>
+
+       PR c++/83937 - wrong C++17 handling of init-list ctor argument.
+       * call.c (build_special_member_call): Don't convert an init-list
+       argument directly to the class type.
+
 2018-03-16  Jakub Jelinek  <jakub@redhat.com>
 
        PR c++/79937
index 5cee63fed092ee3ea39ffa366e6bb08bf6f5e77e..a32f41915fc00a4dcba119158e9ec71dbaeec58a 100644 (file)
@@ -8827,7 +8827,12 @@ build_special_member_call (tree instance, tree name, vec<tree, va_gc> **args,
        /* If we're using this to initialize a non-temporary object, don't
           require the destructor to be accessible.  */
        sub_complain |= tf_no_cleanup;
-      if (!reference_related_p (class_type, TREE_TYPE (arg)))
+      if (BRACE_ENCLOSED_INITIALIZER_P (arg)
+         && !CONSTRUCTOR_IS_DIRECT_INIT (arg))
+       /* An init-list arg needs to convert to the parm type (83937), so fall
+          through to normal processing.  */
+       arg = error_mark_node;
+      else if (!reference_related_p (class_type, TREE_TYPE (arg)))
        arg = perform_implicit_conversion_flags (class_type, arg,
                                                 sub_complain,
                                                 flags);
diff --git a/gcc/testsuite/g++.dg/cpp0x/initlist-ctor2.C b/gcc/testsuite/g++.dg/cpp0x/initlist-ctor2.C
new file mode 100644 (file)
index 0000000..ebacec0
--- /dev/null
@@ -0,0 +1,15 @@
+// PR c++/83937
+// { dg-do run { target c++11 } }
+
+struct S
+{
+  S(int v = 42) {
+    if (v != int{})
+      __builtin_abort();
+  }
+};
+
+int main()
+{
+  S( {} );
+}