njames93 added inline comments.
================ Comment at: clang/lib/AST/NestedNameSpecifier.cpp:314 + InnerPolicy); + } else if (const auto *SpecType = + dyn_cast<DependentTemplateSpecializationType>(T)) { ---------------- Can this be renamed as it shadows the `SpecType` variable from line 305, maybe `DepSpecType`. ================ Comment at: clang/lib/AST/TypePrinter.cpp:1391-1392 T->getQualifier()->print(OS, Policy); + OS << "template "; OS << T->getIdentifier()->getName(); printTemplateArgumentList(OS, T->template_arguments(), Policy); ---------------- This can be merged to one statement. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D78139/new/ https://reviews.llvm.org/D78139 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits