[clang] [clang] Remove the EnableLifetimeWarnings flag in lifetime analysis. (PR #105884)

Haojian Wu via cfe-commits cfe-commits at lists.llvm.org
Fri Aug 23 13:16:23 PDT 2024


================
@@ -393,11 +392,9 @@ static void visitFunctionCallArguments(IndirectLocalPath &Path, Expr *Call,
     Path.push_back({IndirectLocalPathEntry::LifetimeBoundCall, Arg, D});
     if (Arg->isGLValue())
       visitLocalsRetainedByReferenceBinding(Path, Arg, RK_ReferenceBinding,
-                                            Visit,
-                                            /*EnableLifetimeWarnings=*/false);
+                                            Visit);
     else
-      visitLocalsRetainedByInitializer(Path, Arg, Visit, true,
-                                       /*EnableLifetimeWarnings=*/false);
----------------
hokein wrote:

We change the behavior on this line and the line above. (Previously, we never run the GSL code path for the `Arg`).

The fix is on Line 409. 

https://github.com/llvm/llvm-project/pull/105884


More information about the cfe-commits mailing list