[libcxx-commits] [PATCH] D134830: [libc++] Simplify transitive includes test and fix a bug

[Louis Dionne via Phabricator via libcxx-commits]

ldionne created this revision.
Herald added a project: All.
ldionne requested review of this revision.
Herald added a project: libc++.
Herald added a subscriber: libcxx-commits.
Herald added a reviewer: libc++.

This patch incorporates the "sanitize" step of the transitive includes
test into the CSV generator itself. In doing so, it removes complexity
in the test but also fixes a bug where we would filter out <__mutex>
from the output, leading to an incorrect list of includes for the
<shared_mutex> header.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D134830

Files:
  libcxx/test/libcxx/transitive_includes.sanitize.py
  libcxx/test/libcxx/transitive_includes.sh.cpp
  libcxx/test/libcxx/transitive_includes/cxx14.csv
  libcxx/test/libcxx/transitive_includes/cxx17.csv
  libcxx/test/libcxx/transitive_includes/cxx20.csv
  libcxx/test/libcxx/transitive_includes/cxx2b.csv
  libcxx/test/libcxx/transitive_includes_to_csv.py

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D134830.463668.patch
Type: text/x-patch
Size: 54630 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/libcxx-commits/attachments/20220928/90144769/attachment-0001.bin>