[PATCH] D120540: [Driver] Enable to use C++20 modules standalone by -fcxx-modules

[Iain Sandoe via Phabricator via cfe-commits]

iains added a comment.

I guess Chuanqi's TZ is in sleep mode at the moment, so the revert makes sense.

Please let's not use -fmodules-ts to mean anything - we want to phase it out and make it a NOP (we are implementing the standardised modules; I do not see anyone putting effort into finishing the ts implementation).

my suggestion is to introduce `-fmodules={cxx20, clang, etc. etc}` where `-fmodules` is mapped in the driver to `-fmodules=clang` ... and to introduce an enumeration in the options that is visible in the FE so that we can be explicit in checking **which** kind of modules we mean at each stage.

unfortunately, right now I cannot volunteer to implement the suggestion ...


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D120540