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

Reply via email to