[PATCH] D139985: [clang][AST][NFC] Move isSubstitutedDefaultArgument out of TypePrinter into separate component
Adrian Prantl via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Dec 14 09:28:14 PST 2022
aprantl added a comment.
Seems fine to me, I just have one inline comment about the name of the namespace.
================
Comment at: clang/include/clang/AST/TemplateUtils.h:19
+namespace clang {
+namespace TemplateUtils {
+/// Make a best-effort determination of whether the type T can be produced by
----------------
It looks like all the other namespaces in Clang are all lowercase?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D139985/new/
https://reviews.llvm.org/D139985
More information about the cfe-commits
mailing list