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

Tue Jul 14 06:15:40 PDT 2020

Szelethus added inline comments.
Herald added subscribers: ASDenysPetrov, martong.

================
Comment at: clang/include/clang/Analysis/IssueHash.h:35
+///
 /// In case a new hash is introduced, the old one should still be maintained for
 /// a while. One should not introduce a new hash for every change, it is
----------------
gribozavr wrote:
> I don't understand what this paragraph is talking about at all. What does it mean for a new hash to be introduced? As in, when this hashing algorithm changes?
It might refer to the extreme headaches issue hash changes can cause. In D77866,a block of comments in `MallocChecker.cpp` talks about this, and some other discussion (D77866#2068394) in the patch as well.

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

https://reviews.llvm.org/D67421