Skip to content

Commit

Permalink
[clang] Add test for CWG600
Browse files Browse the repository at this point in the history
P1787: //CWG600 is resolved by explaining that accessibility affects naming a member in the sense of the ODR.//
Wording: see changes to [class.access] p1 and p4.
Additional references: [[ http://eel.is/c++draft/basic.def.odr#8.sentence-2 | basic.def.odr/8 ]]: //A function is odr-used if it is named by a potentially-evaluated expression or conversion.//

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

Differential Revision: https://reviews.llvm.org/D139173
  • Loading branch information
Endilll committed Dec 6, 2022
1 parent 2fbe3f9 commit 7e31d07
Show file tree
Hide file tree
Showing 2 changed files with 17 additions and 1 deletion.
16 changes: 16 additions & 0 deletions clang/test/CXX/drs/dr6xx.cpp
Expand Up @@ -3,6 +3,22 @@
// RUN: %clang_cc1 -std=c++14 %s -verify -fexceptions -fcxx-exceptions -pedantic-errors -fno-spell-checking
// RUN: %clang_cc1 -std=c++17 %s -verify -fexceptions -fcxx-exceptions -pedantic-errors -fno-spell-checking
// RUN: %clang_cc1 -std=c++20 %s -verify -fexceptions -fcxx-exceptions -pedantic-errors -fno-spell-checking
// RUN: %clang_cc1 -std=c++2b %s -verify -fexceptions -fcxx-exceptions -pedantic-errors -fno-spell-checking

namespace dr600 { // dr600: yes
struct S {
void f(int);

private:
void f(double); // expected-note {{declared private here}}
};

void g(S *sp) {
sp->f(2);
// access control is applied after overload resolution
sp->f(2.2); // expected-error {{is a private member}}
}
} // namespace dr600

namespace std {
struct type_info {};
Expand Down
2 changes: 1 addition & 1 deletion clang/www/cxx_dr_status.html
Expand Up @@ -3641,7 +3641,7 @@ <h2 id="cxxdr">C++ defect report implementation status</h2>
<td><a href="https://wg21.link/cwg600">600</a></td>
<td>CD6</td>
<td>Does access control apply to members or to names?</td>
<td class="none" align="center">Unknown</td>
<td class="full" align="center">Yes</td>
</tr>
<tr id="601">
<td><a href="https://wg21.link/cwg601">601</a></td>
Expand Down

0 comments on commit 7e31d07

Please sign in to comment.