hamzasood marked 6 inline comments as done. hamzasood added inline comments.
================ Comment at: lib/AST/DeclPrinter.cpp:107-108 - void printTemplateParameters(const TemplateParameterList *Params); + void printTemplateParameters(const TemplateParameterList *Params, + bool OmitTemplateKW = false); void printTemplateArguments(const TemplateArgumentList &Args, ---------------- rsmith wrote: > Nit: I'd prefer splitting this into two functions (one that prints > 'template', calls the other, then prints a space, and the other to print the > actual template parameters) rather than passing a boolean flag. Do you reckon that the boolean flag is okay for `TemplateParameterList::print`? I've left this change for now until that has been decided. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D36527/new/ https://reviews.llvm.org/D36527 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits