[PATCH] D78000: [ASTImporter] Fix handling of not defined FromRecord in ImportContext(...)

Shafik Yaghmour via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Apr 21 15:11:57 PDT 2020


shafik updated this revision to Diff 259110.
shafik marked 2 inline comments as done.
shafik added a comment.

- Simplifying the changes in `ASTImporter::ImportContext`
- Removing `DC->setHasExternalLexicalStorage(true);` from unit test since we are checking `getExternalSource()`


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D78000/new/

https://reviews.llvm.org/D78000

Files:
  clang/lib/AST/ASTImporter.cpp
  clang/unittests/AST/ASTImporterTest.cpp
  lldb/test/API/commands/expression/import_base_class_when_class_has_derived_member/TestImportBaseClassWhenClassHasDerivedMember.py
  lldb/test/API/commands/expression/import_base_class_when_class_has_derived_member/main.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D78000.259110.patch
Type: text/x-patch
Size: 6439 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20200421/67c2b874/attachment.bin>


More information about the cfe-commits mailing list