[libcxx-commits] [PATCH] D102781: [libc++] Alphabetize header inclusions. NFCI

[Raphael Isemann via Phabricator via libcxx-commits]

teemperor added a comment.

I wonder if the `stds_include.sh.cpp` should also use the `-fmodules-local-submodule-visibility` flag. The default `-fmodules` can't really handle top modules that include textual headers. We're using the same flag  in LLVM to build since D74892 <https://reviews.llvm.org/D74892>  (see also rG82b47b2978405f802a33b00d046e6f18ef6a47be <https://reviews.llvm.org/rG82b47b2978405f802a33b00d046e6f18ef6a47be> for why)


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D102781/new/

https://reviews.llvm.org/D102781