[PATCH] D10305: [Clang Static Analyzer] Bug identification
Phillip Power via cfe-commits
cfe-commits at lists.llvm.org
Fri Oct 16 04:48:34 PDT 2015
phillip.power added a comment.
In http://reviews.llvm.org/D10305#262534, @xazax.hun wrote:
> - Should we require the generation of old hashes once a change is introduced, or should we expect users who rely on old hash to maintain the old hash generation as an out of tree patch?
I will likely release the analyzer with all the previous hashes generated by default. I am happy to enable old hashes out of tree, as long as enabling is a small change.
In http://reviews.llvm.org/D10305#262534, @xazax.hun wrote:
> - The hash calculation WILL change in the near future once we figured out how to identify checkers properly (but I think it will not make sense to rename the hash for this change). For this reason I think we should mark this feature as experimental, until that change is introduced. What is the recommended way, to do that? Generating a comment to the plist? Just adding a comment to the headers? Only mention it in the commit log?
How close is "the near future"? I would like to start using the hashing feature in the next couple of weeks. If your checker identification improvements are a long time out, I would like you to submit the current hash as non-experimental.
Best regards,
Phillip
SN Systems - Sony Computer Entertainment
http://reviews.llvm.org/D10305
More information about the cfe-commits
mailing list