asavonic wrote: > Our idea is summarized in > https://discourse.llvm.org/t/rfc-lldb-more-reliable-completion-of-record-types/77442. > Basically the goal is to guarantee that a call to `getDefinition`, _will_ > fetch the definition. This is something that Clang already does, but we just > never implement (via `CompleteRedeclChain`). You're right that the "minimal > import" mode was implemented for perf. reasons. Though we should probably > revisit this. I think we can make LLDB use non-minimal import for record > types while keeping the number of transitive imports contained. It would > still require changes to the importer to be smarter about what we import, but > we wouldn't have these "complete but incomplete" types floating around.
Thanks a lot Michael. It's a lot of work to get the redesign ready, considering both functional and performance requirements. In the meantime, should we merge this patch to fix at least some current problems? https://github.com/llvm/llvm-project/pull/107828 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits