================ @@ -280,7 +280,8 @@ std::optional<Decl *> CxxModuleHandler::tryInstantiateStdTemplate(Decl *d) { new_class_template->getDeclContext(), new_class_template->getTemplatedDecl()->getLocation(), new_class_template->getLocation(), new_class_template, imported_args, - nullptr); + td->hasMatchedPackOnParmToNonPackOnArg(), + /*PrevDecl=*/nullptr); ---------------- mizvekov wrote:
@DavidSpickett @Michael137 this is the only change for this relanding of the patch. As far as I understood reading this code, this is basically importing a template instantiation into a different context, so this is much like what we would do at the AST Importer, ie just forward the new bool from the original specialization into this new one. Also as I understand, this implementation whitelists only a few template names from the STL, none of which have template template parameter to begin with, so this new flag will always be false here and this is pretty much untestable. https://github.com/llvm/llvm-project/pull/125791 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits