[PATCH] D58612: Use std::atomic<> for static counters.
Bruno Ricci via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Feb 25 07:28:45 PST 2019
riccibruno added a comment.
In D58612#1408991 <https://reviews.llvm.org/D58612#1408991>, @alexfh wrote:
> In D58612#1408942 <https://reviews.llvm.org/D58612#1408942>, @riccibruno wrote:
> > Okay, but what about the other similar uses of static members which have the same problem ?
> Do you have any example in mind? I've only seen TSan warnings for these counters, nothing else so far.
For example in `DeclBase.cpp`
#define DECL(DERIVED, BASE) static int n##DERIVED##s = 0;
which count the number of declaration node of each kind.
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
More information about the cfe-commits