bkramer accepted this revision.
bkramer added a comment.
This revision is now accepted and ready to land.
The reason why we disallowed specializations is that there are type-trait
classes that get specialized all over the place (DenseMapInfo in LLVM is one
example). I think we can get away with allowing partial specializations though.
Those should be much less common and we can give the user a choice between the
headers that contain them.
================
Comment at: include-fixer/find-all-symbols/FindAllSymbols.cpp:39
+ if (Node.getTemplateSpecializationKind() == TSK_ExplicitSpecialization) {
+ bool is_partial_specialization =
+ llvm::isa<VarTemplatePartialSpecializationDecl>(Node) ||
----------------
PascalCase for variable names.
https://reviews.llvm.org/D27920
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
