[PATCH] D17214: Stop using "template" when printing qualtype names

Reid Kleckner via cfe-commits cfe-commits at lists.llvm.org
Fri Feb 12 14:09:59 PST 2016


rnk added a subscriber: rnk.
rnk accepted this revision.
rnk added a reviewer: rnk.
rnk added a comment.
This revision is now accepted and ready to land.

lgtm

This code seems like it's intended to avoid operating on dependent types, so it shouldn't need the template keyword. It's hard to have a fully qualified name that's valid at global scope and still somehow mentions template parameters.


================
Comment at: unittests/Tooling/QualTypeNamesTest.cpp:138
@@ -136,3 +137,3 @@
       "  typedef int non_dependent_type;\n"
       "  dependent_type dependent_type_var;\n"
       "  non_dependent_type non_dependent_type_var;\n"
----------------
What's supposed to be printed for dependent_type_var?


http://reviews.llvm.org/D17214





More information about the cfe-commits mailing list