[PATCH] D121588: [C++20][Modules][Driver][HU 1/N] Initial handling for -xc++-{system,user}-header.

Iain Sandoe via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Mar 17 18:02:05 PDT 2022


iains added a comment.

In D121588#3390956 <https://reviews.llvm.org/D121588#3390956>, @vsapsai wrote:

> Thanks for explaining the desired interplay between PCH and PCM. The problem I'm seeing is that it can be hard to find correct compiler flags to achieve the desired result (what is enabled and what is disabled). But that is out of scope for this change.

IMHO, that is an unfortunate consequence of having ≈ 60+ module-related command line flags; we do need to rationalise this.  This is being discussed elsewhere [D120540 <https://reviews.llvm.org/D120540>] (and probably it's better to leave that conversation in one place).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D121588



More information about the cfe-commits mailing list