[libcxx-commits] [PATCH] D148927: [libc++] Adds more forward declaration headers.

Mark de Wever via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Mon Apr 24 23:20:19 PDT 2023


Mordante created this revision.
Herald added a subscriber: arichardson.
Herald added a project: All.
Mordante updated this revision to Diff 515753.
Mordante added a comment.
Mordante updated this revision to Diff 515823.
Mordante updated this revision to Diff 515847.
Mordante updated this revision to Diff 515860.
Mordante updated this revision to Diff 516049.
Mordante updated this revision to Diff 516060.
Mordante published this revision for review.
Herald added a project: libc++.
Herald added a subscriber: libcxx-commits.
Herald added a reviewer: libc++.

CI fixes.


Mordante added a comment.

CI fixes.


Mordante added a comment.

Trigger CI.


Mordante added a comment.

Retrigger CI.


Mordante added a comment.

Retrigger CI.


Mordante added a comment.

CI fixes.


The module validation script of D144994 <https://reviews.llvm.org/D144994> validate whether the contents of
an include match its module. An include is the set of files matching the
pattern:

- foo
- foo/*.
- __fwd/foo.h

Several declarations of the stream headers are in the header iosfwd.
This gives issue using the validation script. Adding iosfwd to the set
of matching files gives too many declarations. For example when
validating the fstream header it will pull in declarations of the
istream header. Instead if writing a set of filters the headers are
granularized into smaller headers containing the expected declarations.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D148927

Files:
  libcxx/include/CMakeLists.txt
  libcxx/include/__fwd/fstream.h
  libcxx/include/__fwd/ios.h
  libcxx/include/__fwd/istream.h
  libcxx/include/__fwd/ostream.h
  libcxx/include/__fwd/sstream.h
  libcxx/include/__fwd/streambuf.h
  libcxx/include/fstream
  libcxx/include/ios
  libcxx/include/iosfwd
  libcxx/include/istream
  libcxx/include/module.modulemap.in
  libcxx/include/ostream
  libcxx/include/sstream
  libcxx/include/streambuf
  libcxx/test/libcxx/private_headers.verify.cpp
  libcxx/utils/data/ignore_format.txt

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D148927.516060.patch
Type: text/x-patch
Size: 24074 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/libcxx-commits/attachments/20230425/c06b3128/attachment-0001.bin>


More information about the libcxx-commits mailing list