[llvm] 905f34e - [NFCI][SanCov] Remove unused default arguments

Vitaly Buka via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 19 18:48:02 PDT 2024


Author: Vitaly Buka
Date: 2024-04-19T18:34:45-07:00
New Revision: 905f34eeca628da526293a21daa6a645012219c6

URL: https://github.com/llvm/llvm-project/commit/905f34eeca628da526293a21daa6a645012219c6
DIFF: https://github.com/llvm/llvm-project/commit/905f34eeca628da526293a21daa6a645012219c6.diff

LOG: [NFCI][SanCov] Remove unused default arguments

Added: 
    

Modified: 
    llvm/lib/Transforms/Instrumentation/SanitizerCoverage.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Transforms/Instrumentation/SanitizerCoverage.cpp b/llvm/lib/Transforms/Instrumentation/SanitizerCoverage.cpp
index a7192d3d075dab..cd766dd3b42e0f 100644
--- a/llvm/lib/Transforms/Instrumentation/SanitizerCoverage.cpp
+++ b/llvm/lib/Transforms/Instrumentation/SanitizerCoverage.cpp
@@ -209,12 +209,13 @@ class ModuleSanitizerCoverage {
   using DomTreeCallback = function_ref<const DominatorTree &(Function &F)>;
   using PostDomTreeCallback =
       function_ref<const PostDominatorTree &(Function &F)>;
-  ModuleSanitizerCoverage(
-      const SanitizerCoverageOptions &Options = SanitizerCoverageOptions(),
-      const SpecialCaseList *Allowlist = nullptr,
-      const SpecialCaseList *Blocklist = nullptr)
-      : Options(OverrideFromCL(Options)), Allowlist(Allowlist),
+
+  ModuleSanitizerCoverage(const SanitizerCoverageOptions &Options,
+                          const SpecialCaseList *Allowlist,
+                          const SpecialCaseList *Blocklist)
+      : Options(Options), Allowlist(Allowlist),
         Blocklist(Blocklist) {}
+
   bool instrumentModule(Module &M, DomTreeCallback DTCallback,
                         PostDomTreeCallback PDTCallback);
 
@@ -285,7 +286,7 @@ class ModuleSanitizerCoverage {
 
 PreservedAnalyses SanitizerCoveragePass::run(Module &M,
                                              ModuleAnalysisManager &MAM) {
-  ModuleSanitizerCoverage ModuleSancov(Options, Allowlist.get(),
+  ModuleSanitizerCoverage ModuleSancov(OverrideFromCL(Options), Allowlist.get(),
                                        Blocklist.get());
   auto &FAM = MAM.getResult<FunctionAnalysisManagerModuleProxy>(M).getManager();
   auto DTCallback = [&FAM](Function &F) -> const DominatorTree & {


        


More information about the llvm-commits mailing list