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

[Benjamin Kramer via Phabricator via cfe-commits]

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