[LifetimeSafety] Guard debug-only function with NDEBUG

[usx95]

Wrap debug-only function in #ifndef NDEBUG to avoid unused function warnings in release builds.

[usx95]

• [LifetimeSafety] Guard debug-only function with NDEBUG #172696 👈 (View in Graphite)
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[llvmbot]

@llvm/pr-subscribers-clang
@llvm/pr-subscribers-clang-analysis

@llvm/pr-subscribers-clang-temporal-safety

Author: Utkarsh Saxena (usx95)

Changes

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 --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)