forked from OSchip/llvm-project
51e0bbb02d
Summary: As we figured out in D67803, importing declarations from a temporary ASTContext that were originally from a persistent ASTContext causes a bunch of duplicated declarations where we end up having declarations in the target AST that have no associated ASTImporter that can complete them. I haven't figured out how/if we can solve this in the current way we do things in LLDB, but in the modern-type-lookup this is solvable as we have a saner architecture with the ExternalASTMerger. As we can (hopefully) make modern-type-lookup the default mode in the future, I would say we try fixing this issue here. As we don't use the hack that was reinstated in D67803 during modern-type-lookup, the test case for this is essentially just printing any kind of container in `std::` as we would otherwise run into the issue that required a hack like D67803. What this patch is doing in essence is that instead of importing a declaration from a temporary ASTContext, we instead check if the declaration originally came from a persistent ASTContext (e.g. the debug information) and we directly import from there. The ExternalASTMerger is already connected with ASTImporters to these different sources, so this patch is essentially just two parts: 1. Mark our temporary ASTContext/ImporterSource as temporary when we import from the expression AST. 2. If the ExternalASTMerger sees we import from the expression AST, instead of trying to import these temporary declarations, check if we can instead import from the persistent ASTContext that is already connected. This ensures that all records from the persistent source actually come from the persistent source and are minimally imported in a way that allows them to be completed later on in the target AST. The next step is to run the ASTImporter for these temporary expressions with the MinimalImport mode disabled, but that's a follow up patch. This patch fixes most test failures with modern-type-lookup enabled by default (down to 73 failing tests, which includes the 22 import-std-module tests which need special treatment). Reviewers: shafik, martong Reviewed By: martong Subscribers: aprantl, rnkovacs, christof, abidh, JDevlieghere, lldb-commits Tags: #lldb Differential Revision: https://reviews.llvm.org/D68326 llvm-svn: 373711 |
||
|---|---|---|
| .. | ||
| abbreviation | ||
| alias | ||
| archives | ||
| asan | ||
| avoids-fd-leak | ||
| backticks | ||
| breakpoint | ||
| completion | ||
| conditional_break | ||
| darwin_log | ||
| data-formatter | ||
| dead-strip | ||
| deleted-executable | ||
| dynamic_value_child_count | ||
| exec | ||
| fat_archives | ||
| float-display | ||
| gdb_remote_client | ||
| history | ||
| inferior-assert | ||
| inferior-changed | ||
| inferior-crashing | ||
| inline-stepping | ||
| jitloader_gdb | ||
| load_unload | ||
| load_using_paths | ||
| longjmp | ||
| memory | ||
| memory-region | ||
| modern-type-lookup | ||
| mtc/simple | ||
| multidebugger_commands | ||
| multiword-commands | ||
| non-overlapping-index-variable-i | ||
| object-file | ||
| param_entry_vals/basic_entry_values_x86_64 | ||
| paths | ||
| plugins | ||
| postmortem | ||
| pre_run_dylibs | ||
| process_group | ||
| process_save_core | ||
| ptr_refs | ||
| recursion | ||
| rerun | ||
| return-value | ||
| set-data | ||
| show_location | ||
| signal | ||
| source-map | ||
| stats_api | ||
| step-avoids-no-debug | ||
| step_scripted | ||
| tail_call_frames | ||
| target-new-solib-notifications | ||
| target_var | ||
| testid | ||
| thread | ||
| tsan | ||
| tty | ||
| type_completion | ||
| type_lookup | ||
| ubsan | ||
| unwind | ||
| value_md5_crash | ||
| var_path | ||
| wrong_commands | ||