[clang][SSAF] Add missing explicit to single-argument constructors (#193052)

This PR adds `explicit` to `TUSummary`,
`UnsafeBufferUsageTUSummaryExtractor`, and
`UnsafeBufferUsageEntitySummary` constructors. This ensures uniform use
of `explicit` for all SSAF single-argument constructors.

Author: aviralg

clang/include/clang/ScalableStaticAnalysisFramework/Analyses/UnsafeBufferUsage/UnsafeBufferUsage.h

@@ -29,7 +29,7 @@ class UnsafeBufferUsageEntitySummary final : public EntitySummary {
   friend llvm::iterator_range<EntityPointerLevelSet::const_iterator>
   getUnsafeBuffers(const UnsafeBufferUsageEntitySummary &);
 
-  UnsafeBufferUsageEntitySummary(EntityPointerLevelSet UnsafeBuffers)
+  explicit UnsafeBufferUsageEntitySummary(EntityPointerLevelSet UnsafeBuffers)
       : EntitySummary(), UnsafeBuffers(std::move(UnsafeBuffers)) {}
 
 public:

clang/include/clang/ScalableStaticAnalysisFramework/Analyses/UnsafeBufferUsage/UnsafeBufferUsageExtractor.h

@@ -19,7 +19,7 @@
 namespace clang::ssaf {
 class UnsafeBufferUsageTUSummaryExtractor : public TUSummaryExtractor {
 public:
-  UnsafeBufferUsageTUSummaryExtractor(TUSummaryBuilder &Builder)
+  explicit UnsafeBufferUsageTUSummaryExtractor(TUSummaryBuilder &Builder)
       : TUSummaryExtractor(Builder) {}
 
   EntityId addEntity(EntityName EN) { return SummaryBuilder.addEntity(EN); }

clang/include/clang/ScalableStaticAnalysisFramework/Core/TUSummary/TUSummary.h

@@ -33,7 +33,8 @@ class TUSummary {
       Data;
 
 public:
-  TUSummary(BuildNamespace TUNamespace) : TUNamespace(std::move(TUNamespace)) {}
+  explicit TUSummary(BuildNamespace TUNamespace)
+      : TUNamespace(std::move(TUNamespace)) {}
 
   friend class SerializationFormat;
   friend class TestFixture;

clang/unittests/ScalableStaticAnalysisFramework/Analyses/CallGraph/CallGraphExtractorTest.cpp

@@ -123,8 +123,8 @@ template <typename... Matchers> auto hasSummaryThat(const Matchers &...Ms) {
 static const SummaryName CallGraphName{CallGraphSummary::Name.str()};
 
 struct CallGraphExtractorTest : ssaf::TestFixture {
-  TUSummary Summary =
-      BuildNamespace(BuildNamespaceKind::CompilationUnit, "Mock.cpp");
+  TUSummary Summary{
+      BuildNamespace(BuildNamespaceKind::CompilationUnit, "Mock.cpp")};
   TUSummaryBuilder Builder = TUSummaryBuilder(Summary);
 
   /// Creates the AST and extractor, then extracts the summaries from the AST.

clang/unittests/ScalableStaticAnalysisFramework/Registries/SummaryExtractorRegistryTest.cpp

@@ -21,7 +21,8 @@ using namespace ssaf;
 
 [[nodiscard]]
 static TUSummary makeFakeSummary() {
-  return BuildNamespace(BuildNamespaceKind::CompilationUnit, "Mock.cpp");
+  return TUSummary(
+      BuildNamespace(BuildNamespaceKind::CompilationUnit, "Mock.cpp"));
 }
 
 namespace {

clang/unittests/ScalableStaticAnalysisFramework/TUSummaryBuilderTest.cpp

@@ -91,8 +91,8 @@ void PrintTo(const MockSummaryData3 &S, std::ostream *OS) {
 }
 
 struct TUSummaryBuilderTest : ssaf::TestFixture {
-  TUSummary Summary =
-      BuildNamespace(BuildNamespaceKind::CompilationUnit, "Mock.cpp");
+  TUSummary Summary{
+      BuildNamespace(BuildNamespaceKind::CompilationUnit, "Mock.cpp")};
   TUSummaryBuilder Builder = TUSummaryBuilder(this->Summary);
 
   [[nodiscard]] static SmallVector<SummaryName>