[PATCH] D53856: [analyzer] Put llvm.Conventions back in alpha
Henry Wong via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Oct 29 19:47:34 PDT 2018
MTC added a comment.
In addition, `clang/lib/StaticAnalyzer/README.txt` and other related docs in `clang/lib/docs/analyzer` are also out of date.
================
Comment at: www/analyzer/alpha_checks.html:570
+<ul>
+ <li>A <code>StringRef</code> should not be bound to a temporary std::string
+ whose lifetime is shorter than the <code>StringRef</code>'s.</li>
----------------
Like `StringRef`, `ArrayRef` may have similar issues. After this patch landed, maybe we can enhance this checker step by step. By that time, we may need the ideas of the heavy users of the LLVM library.
https://reviews.llvm.org/D53856
More information about the cfe-commits
mailing list