Merge branch 'f-ClangTidyFixIfdsIdeHelpers' into f-ClangTidyFixIDEXTaint

Author: Martin Mory

include/phasar/PhasarLLVM/DataFlowSolver/IfdsIde/IFDSFieldSensTaintAnalysis/FlowFunctions/MapTaintedValuesToCaller.h

@@ -20,8 +20,7 @@ class MapTaintedValuesToCaller : public FlowFunction<ExtendedValue> {
   MapTaintedValuesToCaller(const llvm::CallInst *CallInst,
                            const llvm::ReturnInst *RetInst,
                            TraceStats &TraceStats, const ExtendedValue &ZV)
-      : CallInst(CallInst), RetInst(RetInst), TraceStats(TraceStats),
-        ZV(ZV) {}
+      : CallInst(CallInst), RetInst(RetInst), TraceStats(TraceStats), ZV(ZV) {}
   ~MapTaintedValuesToCaller() override = default;
 
   std::set<ExtendedValue> computeTargets(ExtendedValue Fact) override;

include/phasar/PhasarLLVM/DataFlowSolver/IfdsIde/IFDSFieldSensTaintAnalysis/Utils/Log.h

@@ -10,9 +10,8 @@
 #define LOG_INFO(x)                                                            \
   do {                                                                         \
     llvm::outs() << "[ENV_TRACE] " << x << "\n"; /*NOLINT*/                    \
-    \
-    llvm::outs()                                                               \
-        .flush();                                                              \
+                                                                               \
+    llvm::outs().flush();                                                      \
   } while (0)
 
 #ifdef DEBUG_BUILD

include/phasar/PhasarLLVM/DataFlowSolver/Mono/Contexts/CallStringCTX.h

@@ -99,4 +99,4 @@ template <typename N, unsigned K> struct hash<psr::CallStringCTX<N, K>> {
 
 } // namespace std
 
-#endif
+#endif

lib/PhasarLLVM/ControlFlow/Resolver/Resolver.cpp

@@ -77,7 +77,7 @@ Resolver::Resolver(ProjectIRDB &IRDB, LLVMTypeHierarchy &TH)
 
 const llvm::Function *
 Resolver::getNonPureVirtualVFTEntry(const llvm::StructType *T, unsigned Idx,
-                                    const llvm::CallBase */*CallSite*/) {
+                                    const llvm::CallBase * /*CallSite*/) {
   if (TH && TH->hasVFTable(T)) {
     const auto *Target = TH->getVFTable(T)->getFunction(Idx);
     if (Target && Target->getName() != "__cxa_pure_virtual") {

lib/PhasarLLVM/Plugins/IDETabulationProblemTestPlugin.h

@@ -84,4 +84,5 @@ makeIDETabulationProblemtestPlugin(const ProjectIRDB *IRDB,
                                    std::set<std::string> EntryPoints);
 
 } // namespace psr
-#endif
+
+#endif

unittests/PhasarLLVM/ControlFlow/LLVMBasedICFGGlobCtorDtorTest.cpp

@@ -455,7 +455,8 @@ TEST(LLVMBasedICFGGlobCtorDtorTest, LCATest5) {
 
   IDESolver Solver(Problem);
 
-  const auto *GlobalDtor = ICFG.getRegisteredDtorsCallerOrNull(IRDB.getWPAModule());
+  const auto *GlobalDtor =
+      ICFG.getRegisteredDtorsCallerOrNull(IRDB.getWPAModule());
 
   ASSERT_NE(nullptr, GlobalDtor);
 

unittests/PhasarLLVM/DataFlowSolver/IfdsIde/EdgeFunctionSingletonFactoryTest.cpp

@@ -19,7 +19,8 @@ struct TestEdgeFunction
 
   int computeTarget(int /*Source*/) override { return 42; }
 
-  EdgeFunctionPtrType composeWith(EdgeFunctionPtrType /*SecondFunction*/) override {
+  EdgeFunctionPtrType
+  composeWith(EdgeFunctionPtrType /*SecondFunction*/) override {
     return this->shared_from_this();
   };