[all-commits] [llvm/llvm-project] 87d56c: [libc++][modules] Guard carved-out headers more co...

[Louis Dionne via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 87d56c59f52d033cd7c46d769338b9c47fea4929
      https://github.com/llvm/llvm-project/commit/87d56c59f52d033cd7c46d769338b9c47fea4929
  Author: Louis Dionne <ldionne.2 at gmail.com>
  Date:   2024-09-16 (Mon, 16 Sep 2024)

  Changed paths:
    M libcxx/include/__chrono/formatter.h
    M libcxx/include/__chrono/ostream.h
    M libcxx/include/__chrono/parser_std_format_spec.h
    M libcxx/include/__ostream/basic_ostream.h
    M libcxx/include/__ostream/print.h
    M libcxx/include/__thread/jthread.h
    M libcxx/include/__thread/this_thread.h
    M libcxx/include/__thread/thread.h
    M libcxx/include/fstream
    M libcxx/include/iomanip
    M libcxx/include/istream
    M libcxx/include/ostream
    M libcxx/include/sstream
    M libcxx/include/streambuf
    M libcxx/include/syncstream

  Log Message:
  -----------
  [libc++][modules] Guard carved-out headers more consistently (#108637)

Several headers that should not be provided when localization or threads
are disabled were not guarded. That works until one tries to build with
modules and these headers get pulled in.

Note that this could be cleaned up further into something more
systematic, but this patch solves the immediate problems I ran into with
the monolithic modulemap and doesn't create any new inconsistency that
wasn't already there.



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