[PATCH] D85431: [analyzer] Implement a new checker ThreadPrimitivesChecker
Denys Petrov via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Aug 11 01:39:33 PDT 2020
ASDenysPetrov added a comment.
@NoQ
> Umm, why don't you extend `PthreadLockChecker` instead?
First of all I wanted to try to go through all the steps, since it's my first checker. The second is that I considered `PthreadLockChecker` designed for C-functions. Another reason is that it better helps me to test the code when it is separated from all other.
When I feel more confident and it makes sence I'll try to merge this patch into PthreadLockChecker. For now it's more just an idea to discuss, to collect expertise.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D85431/new/
https://reviews.llvm.org/D85431
More information about the cfe-commits
mailing list