public inbox for gcc-cvs@sourceware.org
help / color / mirror / Atom feed
* [gcc r13-1207] c++: class scope function lookup [PR105908]
@ 2022-06-22 22:01 Jason Merrill
  0 siblings, 0 replies; only message in thread
From: Jason Merrill @ 2022-06-22 22:01 UTC (permalink / raw)
  To: gcc-cvs

https://gcc.gnu.org/g:349a39f061407ce2339d2ba25da97005f2030c88

commit r13-1207-g349a39f061407ce2339d2ba25da97005f2030c88
Author: Jason Merrill <jason@redhat.com>
Date:   Wed Jun 22 14:57:21 2022 -0400

    c++: class scope function lookup [PR105908]
    
    In r12-1273 for PR91706, I removed the code in get_class_binding that
    stripped BASELINK.  This testcase demonstrates that we still need to strip
    it in outer_binding before putting the overload set in IDENTIFIER_BINDING,
    for compatibility with bindings added directly for declarations.
    
            PR c++/105908
    
    gcc/cp/ChangeLog:
    
            * name-lookup.cc (outer_binding): Strip BASELINK.
    
    gcc/testsuite/ChangeLog:
    
            * g++.dg/cpp0x/trailing16.C: New test.

Diff:
---
 gcc/cp/name-lookup.cc                   |  4 ++++
 gcc/testsuite/g++.dg/cpp0x/trailing16.C | 17 +++++++++++++++++
 2 files changed, 21 insertions(+)

diff --git a/gcc/cp/name-lookup.cc b/gcc/cp/name-lookup.cc
index 421bf2e4f7a..ce622761a1a 100644
--- a/gcc/cp/name-lookup.cc
+++ b/gcc/cp/name-lookup.cc
@@ -7629,6 +7629,10 @@ outer_binding (tree name,
 		/* Thread this new class-scope binding onto the
 		   IDENTIFIER_BINDING list so that future lookups
 		   find it quickly.  */
+		if (BASELINK_P (class_binding->value))
+		  /* Don't put a BASELINK in IDENTIFIER_BINDING.  */
+		  class_binding->value
+		    = BASELINK_FUNCTIONS (class_binding->value);
 		class_binding->previous = outer;
 		if (binding)
 		  binding->previous = class_binding;
diff --git a/gcc/testsuite/g++.dg/cpp0x/trailing16.C b/gcc/testsuite/g++.dg/cpp0x/trailing16.C
new file mode 100644
index 00000000000..4feb3f81c27
--- /dev/null
+++ b/gcc/testsuite/g++.dg/cpp0x/trailing16.C
@@ -0,0 +1,17 @@
+// PR c++/105908
+// { dg-do compile { target c++11 } }
+
+struct test
+{
+  template <typename T>
+  int templated_func();
+
+  template <typename T>
+  auto call_templated_func() -> decltype(templated_func<T>());
+};
+
+template <typename T>
+auto test::call_templated_func() -> decltype(templated_func<T>())
+{
+  return templated_func<T>();
+}


^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2022-06-22 22:01 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-06-22 22:01 [gcc r13-1207] c++: class scope function lookup [PR105908] Jason Merrill

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).