re PR c++/56359 (Bogus "error: no matching function for call to ...")
authorJason Merrill <jason@redhat.com>
Fri, 22 Feb 2013 22:24:10 +0000 (17:24 -0500)
committerJason Merrill <jason@gcc.gnu.org>
Fri, 22 Feb 2013 22:24:10 +0000 (17:24 -0500)
PR c++/56359
* call.c (can_convert_arg): Discard access checks.

From-SVN: r196229

gcc/cp/ChangeLog
gcc/cp/call.c
gcc/testsuite/g++.dg/template/access25.C [new file with mode: 0644]

index 8f0872b..6b3aa17 100644 (file)
@@ -1,5 +1,8 @@
 2013-02-22  Jason Merrill  <jason@redhat.com>
 
+       PR c++/56359
+       * call.c (can_convert_arg): Discard access checks.
+
        PR c++/56395
        * tree.c (strip_typedefs): Strip typedefs from TYPENAME_TYPE template
        args.
index 25dfd51..7c41421 100644 (file)
@@ -8765,11 +8765,18 @@ can_convert_arg (tree to, tree from, tree arg, int flags,
 
   /* Get the high-water mark for the CONVERSION_OBSTACK.  */
   p = conversion_obstack_alloc (0);
+  /* We want to discard any access checks done for this test,
+     as we might not be in the appropriate access context and
+     we'll do the check again when we actually perform the
+     conversion.  */
+  push_deferring_access_checks (dk_deferred);
 
   t  = implicit_conversion (to, from, arg, /*c_cast_p=*/false,
                            flags, complain);
   ok_p = (t && !t->bad_p);
 
+  /* Discard the access checks now.  */
+  pop_deferring_access_checks ();
   /* Free all the conversions we allocated.  */
   obstack_free (&conversion_obstack, p);
 
diff --git a/gcc/testsuite/g++.dg/template/access25.C b/gcc/testsuite/g++.dg/template/access25.C
new file mode 100644 (file)
index 0000000..e882a70
--- /dev/null
@@ -0,0 +1,20 @@
+// PR c++/56359
+
+typedef int (*InvocationCallback) (const int &);
+
+template < typename target_t >
+void SetPrototypeMethod (target_t, const char *, InvocationCallback);
+
+class A
+{
+    void Initialize ();
+protected:
+    static int Stop (const int &);
+    void Stop ();  // comment out to make the bug disappear.
+};
+
+void
+A::Initialize ()
+{
+    SetPrototypeMethod (0, "stop", A::Stop);
+}