[PATCH] D138091: [Clang] Fix Sema::ClassifyName so that it classifies EnumConstantDecl as NonType when they are brought into scope via using enum

[Nathan Sidwell via Phabricator via cfe-commits]

urnathan added a comment.

Thanks, I didn;'t know about ClassifyName, and obviously never hit a need to adjust it.  Thanks for fixing.

The comment above says we don't resolve member-access non-overload sets in order to check access.  Do we still get that right for, say,

  class B {
   enum A {E}; 
   static auto F (B *p) { return p->E;} // ok
  };
  auto F (B *p) { return p->E; } // not ok

If so, I guess that comment's not right (any more).


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

https://reviews.llvm.org/D138091