[PATCH] D66686: [LifetimeAnalysis] Make it possible to disable the new warnings

Matthias Gehre via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Aug 24 00:56:36 PDT 2019


mgehre added inline comments.


================
Comment at: clang/lib/Sema/SemaInit.cpp:6651
       visitLocalsRetainedByReferenceBinding(Path, Arg, RK_ReferenceBinding,
-                                            Visit);
+                                            Visit, true);
     else
----------------
Could we have argument comments here `/*EnableLifetimeWarnings=*/true,
and especially below where multiple bool literals are passed?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D66686/new/

https://reviews.llvm.org/D66686





More information about the llvm-commits mailing list