[PATCH] D37180: [modules] Add test for using declaration in classes.

2017-08-29 Thread Raphael Isemann via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL311991: [modules] Add test for using declaration in classes. (authored by teemperor). Changed prior to commit: https://reviews.llvm.org/D37180?vs=112788=113044#toc Repository: rL LLVM

[PATCH] D37180: [modules] Add test for using declaration in classes.

2017-08-27 Thread Vassil Vassilev via Phabricator via cfe-commits
v.g.vassilev accepted this revision. v.g.vassilev added a comment. This revision is now accepted and ready to land. LGTM! I'd reword the commit message, saying that we intended to add a fast path in `DeclContext::removeDecl` which checks if the decl is hidden to avoid addition in the lookup