Page MenuHomePhabricator

[clang-tidy] add concurrency-async-no-new-threads
Needs ReviewPublic

Authored by segoon on Wed, Jan 13, 11:11 AM.

Details

Summary

Checks for functions and types that create new system threads.

The check by default searches for types/functions from the following categories:

  • C++ std
  • Boost.Thread, part of Boost.Compute
  • C11 threads
  • POSIX threads (pthreads)
  • Linux syscalls

Custom functions and types can be added via options.

Diff Detail

Event Timeline

segoon created this revision.Wed, Jan 13, 11:11 AM
segoon requested review of this revision.Wed, Jan 13, 11:11 AM
Eugene.Zelenko added inline comments.
clang-tools-extra/docs/clang-tidy/checks/concurrency-async-no-new-threads.rst
7

Please make first statement same as in Release Notes.

26

Please add information about default value. Same below.

Eugene.Zelenko set the repository for this revision to rG LLVM Github Monorepo.
Eugene.Zelenko added a project: Restricted Project.
Eugene.Zelenko added a subscriber: cfe-commits.
Herald added a project: Restricted Project. · View Herald TranscriptThu, Jan 14, 7:20 AM
segoon updated this revision to Diff 316664.Thu, Jan 14, 8:02 AM
segoon edited the summary of this revision. (Show Details)
  • fix the first document line
  • add default values

Eugene.Zelenko, thanks for the review! fixed

Eugene.Zelenko, njames93, any comments on the patch?