teemperor marked 2 inline comments as done. teemperor added inline comments.
================ Comment at: clang/include/clang/AST/ExternalASTMerger.h:92 ImporterTarget Target; + std::shared_ptr<ASTImporterSharedState> SharedState; ---------------- shafik wrote: > Can you add a comment explaining what this is and why we need it and how it > relates to the `ASTImpoter`. > > It is not obvious just looking the local changes what effect adding this has. Added a comment how we used the shared state in all created ASTImporters, but the exact effects of having the shared state are explained in the class itself (e.g. helping the lookup in some cases). Repository: rL LLVM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D68140/new/ https://reviews.llvm.org/D68140 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits