]> git.ipfire.org Git - thirdparty/gcc.git/commitdiff
re PR c++/65695 (NSDMI calling constexpr constructor with pointer-to-member is not...
authorJason Merrill <jason@redhat.com>
Tue, 14 Apr 2015 16:02:41 +0000 (12:02 -0400)
committerJason Merrill <jason@gcc.gnu.org>
Tue, 14 Apr 2015 16:02:41 +0000 (12:02 -0400)
PR c++/65695
* cvt.c (cp_fold_convert): Avoid wrapping PTRMEM_CST in NOP_EXPR.

From-SVN: r222097

gcc/cp/ChangeLog
gcc/cp/cvt.c
gcc/testsuite/g++.dg/cpp0x/constexpr-ptrmem4.C [new file with mode: 0644]

index 03f6f2a422628e9b9bfa6eab637891984518cdd8..3b166a7607860e40b1947919e46878dd69d57394 100644 (file)
@@ -1,5 +1,8 @@
 2015-04-14  Jason Merrill  <jason@redhat.com>
 
+       PR c++/65695
+       * cvt.c (cp_fold_convert): Avoid wrapping PTRMEM_CST in NOP_EXPR.
+
        PR c++/65721
        * name-lookup.c (do_class_using_decl): Complain about specifying
        the current class even if there are dependent bases.
index d0924f100c56181ad1f1aacaf988cb13c476ce4b..9aa9006f1928025e9db72127dcc87899c6f11eec 100644 (file)
@@ -603,8 +603,20 @@ ignore_overflows (tree expr, tree orig)
 tree
 cp_fold_convert (tree type, tree expr)
 {
-  tree conv = fold_convert (type, expr);
-  conv = ignore_overflows (conv, expr);
+  tree conv;
+  if (TREE_TYPE (expr) == type)
+    conv = expr;
+  else if (TREE_CODE (expr) == PTRMEM_CST)
+    {
+      /* Avoid wrapping a PTRMEM_CST in NOP_EXPR.  */
+      conv = copy_node (expr);
+      TREE_TYPE (conv) = type;
+    }
+  else
+    {
+      conv = fold_convert (type, expr);
+      conv = ignore_overflows (conv, expr);
+    }
   return conv;
 }
 
diff --git a/gcc/testsuite/g++.dg/cpp0x/constexpr-ptrmem4.C b/gcc/testsuite/g++.dg/cpp0x/constexpr-ptrmem4.C
new file mode 100644 (file)
index 0000000..68788ca
--- /dev/null
@@ -0,0 +1,26 @@
+// PR c++/65695
+// { dg-do compile { target c++11 } }
+
+struct Foo;
+
+struct Bar
+{
+    using MemberFuncT = int (Foo::*)();
+
+    MemberFuncT h_;
+    constexpr Bar(MemberFuncT h) : h_{h}
+    {
+    }
+};
+
+struct Foo
+{
+    int test()
+    {
+        return -1;
+    }
+
+    static constexpr Bar bar {&Foo::test};
+};
+
+constexpr Bar Foo::bar;