Skip to content

Commit

Permalink
[clang] Add test for CWG360
Browse files Browse the repository at this point in the history
P1787: "CWG360 is resolved by applying access control to using-declarations."
[[ http://eel.is/c++draft/class.access#general-4.sentence-3 | class.access.general#4 ]]: "When a using-declarator is named, access control is applied to it, not to the declarations that replace it."

Reviewed By: #clang-language-wg, aaron.ballman

Differential Revision: https://reviews.llvm.org/D139090
  • Loading branch information
Endilll committed Dec 1, 2022
1 parent 44f7356 commit 3f950ad
Show file tree
Hide file tree
Showing 2 changed files with 28 additions and 1 deletion.
27 changes: 27 additions & 0 deletions clang/test/CXX/drs/dr3xx.cpp
Expand Up @@ -890,6 +890,33 @@ namespace dr359 { // dr359: yes
};
}

namespace dr360 { // dr360: yes
struct A {
int foo();
int bar();

protected:
int baz();
};

struct B : A {
private:
using A::foo; // #dr360-foo-using-decl
protected:
using A::bar; // #dr360-bar-using-decl
public:
using A::baz; // #dr360-baz-using-decl
};

int main() {
int foo = B().foo(); // expected-error {{is a private member}}
// expected-note@#dr360-foo-using-decl {{declared private here}}
int bar = B().bar(); // expected-error {{is a protected member}}
// expected-note@#dr360-bar-using-decl {{declared protected here}}
int baz = B().baz();
}
} // namespace dr360

// dr362: na
// dr363: na

Expand Down
2 changes: 1 addition & 1 deletion clang/www/cxx_dr_status.html
Expand Up @@ -2200,7 +2200,7 @@ <h2 id="cxxdr">C++ defect report implementation status</h2>
<td><a href="https://wg21.link/cwg360">360</a></td>
<td>CD6</td>
<td>Using-declaration that reduces access</td>
<td class="none" align="center">Unknown</td>
<td class="full" align="center">Yes</td>
</tr>
<tr class="open" id="361">
<td><a href="https://wg21.link/cwg361">361</a></td>
Expand Down

0 comments on commit 3f950ad

Please sign in to comment.