[PATCH] D59692: [ASTImporter] Fix name conflict handling
Shafik Yaghmour via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Jul 25 15:33:39 PDT 2019
shafik added a comment.
First round of review.
================
Comment at: clang/lib/AST/ASTImporter.cpp:2632
+ ExpectedName NameOrErr = Importer.HandleNameConflict(
+ Name, DC, IDNS, ConflictingDecls.data(), ConflictingDecls.size());
+ if (!NameOrErr)
----------------
`Name` -> `SearchName`
================
Comment at: clang/unittests/AST/ASTImporterTest.cpp:2392
+struct ImportFunctionTemplates : ASTImporterOptionSpecificTestBase {};
+
----------------
What about tests for name conflicts for:
`NamespaceDecl`
`TypedefNameDecl`
`TypeAliasTemplateDecl`
`EnumConstantDecl`
`RecordDecl`
`VarDecl`
Who were also modified above.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D59692/new/
https://reviews.llvm.org/D59692
More information about the cfe-commits
mailing list