ASTImporter makes now difference between classes with same name in different
translation units if these are not visible outside. These classes are not linked
into one decl chain.
Details
Details
Diff Detail
Diff Detail
- Repository
- rC Clang
- Build Status
Buildable 32383 Build 32382: arc lint + arc unit
Event Timeline
Comment Actions
Thanks for reviewing.
Similar additions are planned to follow with function and variable template, enum, scoped enum, typedef, type alias.
unittests/AST/ASTImporterVisibilityTest.cpp | ||
---|---|---|
34 | The exact decl name is not repeated in these variable names like "FunPattern" for functions. Using "class" instead of "CXXRecord" can be acceptable. I do not like ImportCXXRecordsVisibilityChain (GetRecPattern and ImportRecordsVisibilityChain may be good too, but we do not say "C++ record" for thing that is a class). |