[all-commits] [llvm/llvm-project] 57fe53: [libc++] First attempt to regroup a few modules in...

[Louis Dionne via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 57fe53cae40351ebd079a9a0105addf4ad2e97dd
      https://github.com/llvm/llvm-project/commit/57fe53cae40351ebd079a9a0105addf4ad2e97dd
  Author: Louis Dionne <ldionne.2 at gmail.com>
  Date:   2024-08-30 (Fri, 30 Aug 2024)

  Changed paths:
    M libcxx/include/module.modulemap
    M libcxx/test/libcxx/thread/thread.stoptoken/atomic_unique_lock.pass.cpp
    M libcxx/test/libcxx/thread/thread.stoptoken/intrusive_list_view.pass.cpp
    M libcxx/test/libcxx/thread/thread.stoptoken/intrusive_shared_ptr.pass.cpp

  Log Message:
  -----------
  [libc++] First attempt to regroup a few modules in the modulemap (#98214)

We split up all the headers into top-level modules when we broke up
cycles with the C compatibility headers. However, this resulted in a
large number of small modules, which is awkward and clearly against the
philosophy of Clang modules. This was necessary to make things work.

This patch regroups a few headers from two leaf modules: stop_token and
pstl. It should be pretty uncontroversial that grouping these headers
into a single module doesn't introduce any cyclic dependency, yet it's a
first step towards reducing the number of top-level modules we have in
our modulemap.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications