[PATCH] D62829: [clang-tidy] Check for dynamically initialized statics in headers.

Roman Lebedev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Aug 2 11:49:49 PDT 2019


lebedev.ri added a comment.

I think this last update clarified the concerns i raised in last comment.
But, did you mean to update the tests? They should have broke now that you require `!getLangOpts().ThreadsafeStatics`.



================
Comment at: clang-tools-extra/docs/clang-tidy/checks/bugprone-dynamic-static-initializers.rst:11
+when disabling compiler generated synchronization instructions for
+static variables initialized at runtime (e.g. by -fno-threadsafe-statics), even if a particular project
+takes the necessary precautions to prevent race conditions during
----------------
escape it:
```
``-fno-threadsafe-statics``
```


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

https://reviews.llvm.org/D62829





More information about the cfe-commits mailing list