[PATCH] D100276: [clang] p1099 3/5: using Enum::member

[Nathan Sidwell via Phabricator via cfe-commits]

urnathan marked an inline comment as done.
urnathan added inline comments.


================
Comment at: clang/test/CXX/dcl.dcl/basic.namespace/namespace.udecl/p3.cpp:20
 class C {
+public:
   int g();
----------------
bruno wrote:
> bruno wrote:
> > > The change to clang/test/CXX/dcl.dcl/basic.namespace/namespace.udecl/p3.cpp is to silence an uninteresting access error that the above change causes.
> > 
> > The fact that the access check changed for previous version of the language (not necessarily related to p1099) indicates that this specific change deserves its own patch.
> How about this one?
I didn;t describe this very well.  What happens with the reordering of the checks of the named scopes is that we now diagnose that C::g is private and then that we're reaching into a struct.  Before we just diagnosed the second problem and bailed, never getting to the first.  The test is aiming to discover that second problem is detected, and doesn't care about the access.  Hence make it public.


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

https://reviews.llvm.org/D100276