[all-commits] [llvm/llvm-project] f1417e: [libcxx][module-map] creates submodules for privat...

Christopher Di Bella via All-commits all-commits at lists.llvm.org
Thu Jun 3 11:19:46 PDT 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: f1417eb9b1f51b689c78dd8cb0114c1749dd2845
      https://github.com/llvm/llvm-project/commit/f1417eb9b1f51b689c78dd8cb0114c1749dd2845
  Author: Christopher Di Bella <cjdb at google.com>
  Date:   2021-06-03 (Thu, 03 Jun 2021)

  Changed paths:
    M libcxx/include/__iterator/advance.h
    M libcxx/include/__iterator/indirect_concepts.h
    M libcxx/include/__iterator/next.h
    M libcxx/include/__iterator/prev.h
    M libcxx/include/__memory/allocation_guard.h
    M libcxx/include/__memory/allocator.h
    M libcxx/include/__memory/allocator_traits.h
    M libcxx/include/__memory/pointer_safety.h
    M libcxx/include/__memory/pointer_traits.h
    M libcxx/include/module.modulemap

  Log Message:
  -----------
  [libcxx][module-map] creates submodules for private headers

Most of our private headers need to be treated as submodules so that
Clang modules can export things correctly. Previous commits that split
monolithic headers into smaller chunks were unaware of this requirement,
and so this is being addressed in one fell swoop. Moving forward, most
new headers will need to have their own submodule (anything that's
conditionally included is exempt from this rule, which means `__support`
headers aren't made into submodules).

This hasn't been marked NFC, since I'm not 100% sure that's the case.

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




More information about the All-commits mailing list