[all-commits] [llvm/llvm-project] df51be: [libc++] Split a few utilities out of __threading_...

Louis Dionne via All-commits all-commits at lists.llvm.org
Tue Jan 18 09:23:56 PST 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: df51be85e4aee2365338186e38aebf7ffe81fc90
      https://github.com/llvm/llvm-project/commit/df51be85e4aee2365338186e38aebf7ffe81fc90
  Author: Louis Dionne <ldionne.2 at gmail.com>
  Date:   2022-01-18 (Tue, 18 Jan 2022)

  Changed paths:
    M libcxx/include/CMakeLists.txt
    A libcxx/include/__chrono/convert_to_timespec.h
    A libcxx/include/__thread/timed_backoff_policy.h
    M libcxx/include/__threading_support
    M libcxx/include/atomic
    M libcxx/include/barrier
    M libcxx/include/chrono
    M libcxx/include/module.modulemap
    M libcxx/include/semaphore
    M libcxx/include/thread
    M libcxx/src/atomic.cpp
    A libcxx/test/libcxx/diagnostics/detail.headers/chrono/convert_to_timespec.module.verify.cpp
    A libcxx/test/libcxx/diagnostics/detail.headers/thread/timed_backoff_policy.module.verify.cpp

  Log Message:
  -----------
  [libc++] Split a few utilities out of __threading_support

This change is the basis for a further refactoring where I'm going to
split up the various implementations we have in __threading_support to
make that code easier to understand.

Note that I had to make __convert_to_timespec a template to break
circular dependencies. Concretely, we never seem to use it with anything
other than ::timespec, but I am wary of hardcoding that assumption as
part of this change, since I suspect there's a reason for going through
these hoops in the first place.

Differential Revision: https://reviews.llvm.org/D116944




More information about the All-commits mailing list