[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