[PATCH] D66686: [LifetimeAnalysis] Make it possible to disable the new warnings
Gábor Horváth via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Aug 26 10:02:26 PDT 2019
xazax.hun marked an inline comment as done.
xazax.hun added inline comments.
================
Comment at: clang/lib/Sema/SemaInit.cpp:6651
visitLocalsRetainedByReferenceBinding(Path, Arg, RK_ReferenceBinding,
- Visit);
+ Visit, true);
else
----------------
mgehre wrote:
> Could we have argument comments here `/*EnableLifetimeWarnings=*/true,
> and especially below where multiple bool literals are passed?
Done in r369928.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D66686/new/
https://reviews.llvm.org/D66686
More information about the llvm-commits
mailing list