[PATCH] D147848: [clang] Add test for CWG2370

Shafik Yaghmour via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Apr 10 13:59:11 PDT 2023


shafik added inline comments.


================
Comment at: clang/test/CXX/drs/dr23xx.cpp:182
+  typedef N::type N_type;
+  // FIXME: `type` should be searched for in N
+  // friend void N::g(type);
----------------
The implementation seems to all accept this example: https://godbolt.org/z/vE6bEP6xa

but the examples from the `p1787` have a decidely mixed conformance: https://godbolt.org/z/dhq7oEKaY

but the `A::F(F)` you point out in your example clang does get wrong and gcc does not. So at minimum please file bug reports against the examples that clang does not get right from `p1787` and we need to dig into why your example above seems to not the same since that is what you intended. 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D147848



More information about the cfe-commits mailing list