[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