[clang] [clang][StaticAnalyzer] Add enclosing Decl information to bug reports of RawPtrRef(LocalVars|Member)Checker (PR #214102)

[Donát Nagy via cfe-commits]

NagyDonat wrote:

> I'm a bit unsatisfied with the way of testing this.
> IMO the `getIssueString` should be tested in a unittest, because it's nice and pure.
> How about creating a new unittest as `clang/unittests/Analysis/IssueHashTest.cpp` where we would assert that for a given Decl in a C++ snippet it produces the expected issue hash string.

I personally don't see the value in this more complicated testing. The actual value of a hash is irrelevant (and should not be pinned), the only important property is that we assign different hashes to different bugs -- and that is clearly tested in the existing tests.

https://github.com/llvm/llvm-project/pull/214102