[clang] [SSAF] Let UnsafeBufferUsageExtractor & PointerFlowExtractor ignore templates (PR #198927)

[Balázs Benics via cfe-commits]

================
@@ -34,6 +34,11 @@ class ContributorFinder : public DynamicRecursiveASTVisitor {
 public:
   std::set<const NamedDecl *> Contributors;
 
+  ContributorFinder() {
+    ShouldVisitTemplateInstantiations = true;
+    ShouldVisitImplicitCode = false;
----------------
steakhal wrote:

My personal stance is that if we took the time to evaluate what the default visitation options do then it's probably worths spelling them out explicitly here and not rely on the defaults.
However, this is just my personal preference and I don't see much risk of relying on the defaults.

https://github.com/llvm/llvm-project/pull/198927