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

Tue Mar 15 01:59:50 PDT 2022

iains added a comment.

I am still concerned that there is an expectation that. the fcxx-modules option is connected with clang modules.
.. see, for example:

https://github.com/llvm/llvm-project/blob/d90d45fc9029cc7dbb6d44798f51131df6b2eef1/clang/lib/Driver/ToolChains/Clang.cpp#L3579

and

https://github.com/llvm/llvm-project/blob/875782bd9ea322445dd41213ca6bbf70169e741d/clang/include/clang/Driver/Options.td#L5549

(those are the two places that the option currently appears in the code; test cases that have the option seem to be connected with clang/implicit modules).

So I will defer to other folks to comment further.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D120540