[PATCH] D27920: [find-all-symbols] Index partial template specializations.

Benjamin Kramer via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jan 11 02:37:10 PST 2017

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.


More information about the cfe-commits mailing list