[PATCH] D67421: [analyzer] NFC: Move IssueHash to libAnalysis.

Artem Dergachev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jul 29 21:45:53 PDT 2020


NoQ updated this revision to Diff 281797.
NoQ marked 5 inline comments as done.
NoQ added a comment.

Rebase!

Re-do the comments. Rename `GetIssueHash()` to `getIssueHashV1` because there's a high chance we'll want more different kinds of hashes (also starts with a small letter, while we're at it).


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

https://reviews.llvm.org/D67421

Files:
  clang/include/clang/Analysis/IssueHash.h
  clang/include/clang/StaticAnalyzer/Core/IssueHash.h
  clang/lib/Analysis/CMakeLists.txt
  clang/lib/Analysis/IssueHash.cpp
  clang/lib/StaticAnalyzer/Checkers/ExprInspectionChecker.cpp
  clang/lib/StaticAnalyzer/Core/CMakeLists.txt
  clang/lib/StaticAnalyzer/Core/HTMLDiagnostics.cpp
  clang/lib/StaticAnalyzer/Core/IssueHash.cpp
  clang/lib/StaticAnalyzer/Core/PlistDiagnostics.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D67421.281797.patch
Type: text/x-patch
Size: 12864 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20200730/938e7932/attachment-0001.bin>


More information about the cfe-commits mailing list