]> granicus.if.org Git - clang/commit
[modules] Add test for using declaration in classes.
authorRaphael Isemann <teemperor@gmail.com>
Tue, 29 Aug 2017 09:27:41 +0000 (09:27 +0000)
committerRaphael Isemann <teemperor@gmail.com>
Tue, 29 Aug 2017 09:27:41 +0000 (09:27 +0000)
commit868bd3047d2d770512a3df68c84df509b520b9ce
tree4a4d2f2cd8e61bb735aa9a627bcca15597165819
parentbe837badf3509fd1b93ff9c7259b5bae2f663803
[modules] Add test for using declaration in classes.

Summary:
This adds a test that checks if the using declaration in classes still works as intended with modules.

The motivation for this is that we tried to add a shortcut to `removeDecl` that would skip the removal of declarations from the lookup table if they are hidden. This optimization passed the clang test suite but actually broke the using declaration in combination with -fmodules-local-submodule-visibility. In this mode we hide all decls from other modules such as by chance the parent method, in which case don't remove the parent method from the lookup table and get ambiguous lookup errors. After this patch we now correctly see if this behavior is broken by a patch like this in the test suite.

Reviewers: v.g.vassilev

Reviewed By: v.g.vassilev

Subscribers: cfe-commits

Differential Revision: https://reviews.llvm.org/D37180

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@311991 91177308-0d34-0410-b5e6-96231b3b80d8
test/Modules/using-decl-inheritance.cpp [new file with mode: 0644]