rtrieu added a comment. It looks like a strict improvement on printing and most callers using the default args won't need to be updated.
There's one more function call that should be updated: https://github.com/llvm/llvm-project/blob/main/clang-tools-extra/clangd/DumpAST.cpp#L298 Fixing that and the comment and this should be good to go in. ================ Comment at: clang/include/clang/AST/TemplateName.h:318-320 /// \param SuppressNNS if true, don't print the /// nested-name-specifier that precedes the template name (if it has /// one). ---------------- Update this comment to reflect the enum. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D108794/new/ https://reviews.llvm.org/D108794 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
