================
@@ -3106,7 +3107,8 @@ void clang::sema::AnalysisBasedWarnings::IssueWarnings(
   if (EnableLifetimeSafetyAnalysis && S.getLangOpts().CPlusPlus) {
     if (AC.getCFG()) {
       lifetimes::LifetimeSafetyReporterImpl LifetimeSafetyReporter(S);
-      lifetimes::runLifetimeSafetyAnalysis(AC, &LifetimeSafetyReporter);
+      lifetimes::runLifetimeSafetyAnalysis(
+          AC, &LifetimeSafetyReporter, S.getLangOpts().CfgBlocknumThreshold);
----------------
usx95 wrote:

`AC` already has `AST` which has `getLangOpts()`. Let's remove this extra 
function parameter and directly read the threshold in `run` from the LangOptions

https://github.com/llvm/llvm-project/pull/170444
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to