From 61719b50c5d25ce03abdb9a9ed9e2ccb57496069 Mon Sep 17 00:00:00 2001 From: Bruno Ricci Date: Mon, 22 Apr 2019 11:40:31 +0000 Subject: [Sema][NFC] Add more tests for the behavior of argument-dependent name lookup The goal here is to exercise each rule in [basic.lookup.argdep] at least once. These new tests expose what I believe are 2 issues: 1. CWG 1691 needs to be implemented (p2: [...] Its associated namespaces are the innermost enclosing namespaces of its associated classes [...]) The corresponding tests are adl_class_type::X2 and adl_class_type::X5. 2. The end of paragraph 2 ([...] Additionally, if the aforementioned set of overloaded functions is named with a template-id, its associated classes and namespaces also include those of its type template-arguments and its template template-arguments.) is not implemented. Closely related, the restriction on non-dependent parameter types in this same paragraph needs to be removed. The corresponding tests are in adl_overload_set (both issues are from CWG 997). Differential Revision: https://reviews.llvm.org/D60570 Reviewed By: riccibruno, Quuxplusone git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@358881 91177308-0d34-0410-b5e6-96231b3b80d8 --- .../basic/basic.lookup/basic.lookup.argdep/p4.cpp | 93 ++++++++++++++++++++++ 1 file changed, 93 insertions(+) (limited to 'test/CXX/basic/basic.lookup/basic.lookup.argdep/p4.cpp') diff --git a/test/CXX/basic/basic.lookup/basic.lookup.argdep/p4.cpp b/test/CXX/basic/basic.lookup/basic.lookup.argdep/p4.cpp index 2292fc540c..910eb0c1a8 100644 --- a/test/CXX/basic/basic.lookup/basic.lookup.argdep/p4.cpp +++ b/test/CXX/basic/basic.lookup/basic.lookup.argdep/p4.cpp @@ -67,3 +67,96 @@ namespace test1 { foo(a, 10); // expected-error {{no matching function for call to 'foo'}} } } + + +// Check the rules described in p4: +// When considering an associated namespace, the lookup is the same as the lookup +// performed when the associated namespace is used as a qualifier (6.4.3.2) except that: + +// - Any using-directives in the associated namespace are ignored. +namespace test_using_directives { + namespace M { struct S; } + namespace N { + void f(M::S); // expected-note {{declared here}} + } + namespace M { + using namespace N; + struct S {}; + } + void test() { + M::S s; + f(s); // expected-error {{use of undeclared}} + M::f(s); // ok + } +} + +// - Any namespace-scope friend functions or friend function templates declared in +// associated classes are visible within their respective namespaces even if +// they are not visible during an ordinary lookup +// (Note: For the friend declaration to be visible, the corresponding class must be +// included in the set of associated classes. Merely including the namespace in +// the set of associated namespaces is not enough.) +namespace test_friend1 { + namespace N { + struct S; + struct T { + friend void f(S); // #1 + }; + struct S { S(); S(T); }; + } + + void test() { + N::S s; + N::T t; + f(s); // expected-error {{use of undeclared}} + f(t); // ok, #1 + } +} + +// credit: Arthur O’Dwyer +namespace test_friend2 { + struct A { + struct B { + struct C {}; + }; + friend void foo(...); // #1 + }; + + struct D { + friend void foo(...); // #2 + }; + template struct E { + struct F {}; + }; + + template struct G {}; + template struct H {}; + template struct I {}; + struct J { friend void foo(...) {} }; // #3 + + void test() { + A::B::C c; + foo(c); // #1 is not visible since A is not an associated class + // expected-error@-1 {{use of undeclared}} + E::F f; + foo(f); // #2 is not visible since D is not an associated class + // expected-error@-1 {{use of undeclared}} + G > > j; + foo(j); // ok, #3. + } +} + +// - All names except those of (possibly overloaded) functions and +// function templates are ignored. +namespace test_other_names { + namespace N { + struct S {}; + struct Callable { void operator()(S); }; + static struct Callable Callable; + } + + void test() { + N::S s; + Callable(s); // expected-error {{use of undeclared}} + } +} -- cgit v1.2.3