martong added inline comments.

================
Comment at: lldb/include/lldb/Symbol/StdModuleHandler.h:22
+/// process is setup to use C++ modules and has the 'std' module attached.
+class StdModuleHandler {
+  clang::ASTImporter *m_importer = nullptr;
----------------
I think it would worth to extend this description with something like this:  

`tryInstantiateStdTemplate` directly *creates* (or finds existing) 
specializations from the std module and intercept the import from the debug 
info AST to avoid having broken specializations in the expression evaluation 
context.



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

https://reviews.llvm.org/D59537



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to