[Lldb-commits] [PATCH] D59537: Instantiate 'std' templates explicitly in the expression evaluator

[Raphael Isemann via Phabricator via lldb-commits]

teemperor marked an inline comment as done.
teemperor added inline comments.


================
Comment at: lldb/include/lldb/Symbol/ClangASTImporter.h:249
+    /// it at the end of the scope. Supports being used multiple times on the
+    /// same Minion instance in nested scopes.
+    class CxxModuleScope {
----------------
aprantl wrote:
> teemperor wrote:
> > aprantl wrote:
> > > I really wish we could rename Minion with something actually descriptive...
> > I'm still trying to come up with a good name. The whole purpose of the class is to actually extend and listen to the actual ASTImporter. So what about `ASTImporterDecorator`?
> ASTImporterDelegate perhaps? https://en.wikipedia.org/wiki/Delegation_pattern
Sounds good to me, even though we technically inherit from ASTImporter. But everything is better than Minion :)


Repository:
  rL LLVM

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

https://reviews.llvm.org/D59537