[PATCH] D69933: [ASTImporter] Limit imports of structs

2020-01-21 Thread Greg Clayton via Phabricator via cfe-commits
clayborg added a comment. Clang AST contexts know how to complete types and is done via the external AST source code that will ask a type to complete itself. Each object file has an AST that knows how to lazily complete a type when and only when it is needed. Each object file also only knows ab

[PATCH] D69933: [ASTImporter] Limit imports of structs

2020-01-21 Thread Raphael Isemann via Phabricator via cfe-commits
teemperor added a comment. I really don't think the ASTImporter should ever manipulate records in the source context (effectively the source context should be considered immutable). It also seems *very* wrong that what we import depends in any way on a previous expression so I agree we should f

[PATCH] D69933: [ASTImporter] Limit imports of structs

2020-01-21 Thread Jaroslav Sevcik via Phabricator via cfe-commits
jarin updated this revision to Diff 239235. jarin added a comment. Herald added a subscriber: lldb-commits. I changed the diff so that it does not touch Clang's AST importer, instead it patches LLDB's wrapper of the AST importer. The idea is to only import complete a record if the current evalua

[PATCH] D69933: [ASTImporter] Limit imports of structs

2019-11-07 Thread Jaroslav Sevcik via Phabricator via cfe-commits
jarin created this revision. jarin added a reviewer: martong. jarin added projects: LLDB, clang. Herald added subscribers: cfe-commits, teemperor, rnkovacs. Herald added a reviewer: a.sidorin. Herald added a reviewer: shafik. jarin edited the summary of this revision. This is a work in progress pa