[libcxx-commits] [PATCH] D77697: libc++: adjust modulemap for non-modular C

[Duncan P. N. Exon Smith via Phabricator via libcxx-commits]

dexonsmith added a subscriber: Bigcheese.
dexonsmith added a comment.

In D77697#1969998 <https://reviews.llvm.org/D77697#1969998>, @compnerd wrote:

> @dexonsmith - yeah, sadly I dont think that there is a good way to audit that - any change to the public headers can cause issues.  Furthermore, the libc headers themselves also influence this.


For auditing, can you use `llvm-bcanalyze` to see which headers are claimed by which PCM?  If not, we should probably add a `clang-pcm` tool or something to help inspect module contents.  @Bigcheese, thoughts?

I guess the obvious concern about this is that this is a game of whack-a-mole.  If the headers change, you may need to shuffle module order again, at which point, which version of libc should we make libc++ work against?  And different libc implementations could need different orders.  But I'm just pointing it out; I don't have a problem with this patch landing.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D77697