[PATCH] D139326: [clang] Add test for CWG952

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Dec 5 11:43:46 PST 2022


aaron.ballman accepted this revision.
aaron.ballman added a comment.

LGTM, though I would appreciate another test (that I expect to pass).



================
Comment at: clang/test/CXX/drs/dr9xx.cpp:76
+namespace dr952 { // dr952: yes
+struct A {
+  typedef int I; // #dr952-typedef-decl
----------------
I wouldn't mind another test case:
```
struct A {
protected:
  static int x;
};
struct B : A {
  friend int get(B) { return x; }
};
```
this was something Richard had pointed out needs to still work as part of the reflector discussions of this issue.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D139326/new/

https://reviews.llvm.org/D139326



More information about the cfe-commits mailing list