[PATCH] D90944: [clang-tidy] implement concurrency-mt-unsafe

Roman Lebedev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Nov 26 09:04:30 PST 2020


lebedev.ri added a comment.

In D90944#2418792 <https://reviews.llvm.org/D90944#2418792>, @segoon wrote:

> - move static vectors out of namespace

Nice.

> - mark mt-unsafe decls and check for marks in exprs

Eeeh.
I was thinking of either some smart matcher "match any function declaration
with name from these lists, and then match every call to said decl".
But the current implementation, i'm not sure this approach is even legal for checks.
So i guess let's stick with the original solution..


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

https://reviews.llvm.org/D90944



More information about the cfe-commits mailing list