[Lldb-commits] [PATCH] D62352: Call to HandleNameConflict in VisitRecordDecl mistakeningly using Name instead of SearchName

[Shafik Yaghmour via Phabricator via lldb-commits]

shafik created this revision.
shafik added reviewers: martong, teemperor, jasonmolenda, friss.
Herald added a subscriber: rnkovacs.

https://reviews.llvm.org/D51633 added error handling to the `ASTNodeImporter::VisitRecordDecl` for the conflicting names case. This could lead to erroneous return of an error in that case since we should have been using SearchName. Name may be empty in the case where we find the name via `D->getTypedefNameForAnonDecl()->getDeclName()`.

This fix is very similar to https://reviews.llvm.org/D59665


https://reviews.llvm.org/D62352

Files:
  lib/AST/ASTImporter.cpp


Index: lib/AST/ASTImporter.cpp
===================================================================
--- lib/AST/ASTImporter.cpp
+++ lib/AST/ASTImporter.cpp
@@ -2585,7 +2585,7 @@
     } // for
 
     if (!ConflictingDecls.empty() && SearchName) {
-      Name = Importer.HandleNameConflict(Name, DC, IDNS,
+      Name = Importer.HandleNameConflict(SearchName, DC, IDNS,
                                          ConflictingDecls.data(),
                                          ConflictingDecls.size());
       if (!Name)


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D62352.201085.patch
Type: text/x-patch
Size: 520 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20190523/1d937db3/attachment.bin>