llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-clang-analysis @llvm/pr-subscribers-clang-temporal-safety Author: Utkarsh Saxena (usx95) <details> <summary>Changes</summary> Wrap debug-only function in `#ifndef NDEBUG` to avoid unused function warnings in release builds. --- Full diff: https://github.com/llvm/llvm-project/pull/172696.diff 1 Files Affected: - (modified) clang/lib/Analysis/LifetimeSafety/LifetimeSafety.cpp (+2) ``````````diff diff --git a/clang/lib/Analysis/LifetimeSafety/LifetimeSafety.cpp b/clang/lib/Analysis/LifetimeSafety/LifetimeSafety.cpp index 50111f8ef1fba..c0fa640a8ba66 100644 --- a/clang/lib/Analysis/LifetimeSafety/LifetimeSafety.cpp +++ b/clang/lib/Analysis/LifetimeSafety/LifetimeSafety.cpp @@ -31,6 +31,7 @@ namespace clang::lifetimes { namespace internal { +#ifndef NDEBUG static void DebugOnlyFunction(AnalysisDeclContext &AC, const CFG &Cfg, FactManager &FactMgr) { std::string Name; @@ -43,6 +44,7 @@ static void DebugOnlyFunction(AnalysisDeclContext &AC, const CFG &Cfg, /*ShowColors=*/true)); DEBUG_WITH_TYPE(Name.c_str(), FactMgr.dump(Cfg, AC)); } +#endif LifetimeSafetyAnalysis::LifetimeSafetyAnalysis(AnalysisDeclContext &AC, LifetimeSafetyReporter *Reporter) `````````` </details> https://github.com/llvm/llvm-project/pull/172696 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
