[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