[PATCH] D134267: [C++] [Modules] Support one phase compilation model for named modules

Wed Nov 2 19:31:23 PDT 2022

ChuanqiXu added a comment.

In D134267#3902564 <https://reviews.llvm.org/D134267#3902564>, @ben.boeckel wrote:

> I tried applying this patch to your MyP1689 branch (fixing conflicts with attempts there), but it seems that something isn't being plumbed properly:
>
>   clang-15: error: unknown argument: '-fc++-module-file-output=CMakeFiles/export_bmi_and_interfaces.dir/importable.pcm'
>   clang-15: error: unable to create default module cache path "/home/boeckb/.cache/clang/ModuleCache": No such file or direc
>   tory
>
> There's no way to disable the "module cache path" either (it fails here because I symlinked it to `/dev/null` to avoid things "working" behind my back).

Oh, it is odd. The option `-fc++-module-file-output` should have higher priority than creating module cache path.

BTW, this patch should be discarded. And now we're pursuing D137059 <https://reviews.llvm.org/D137059> and D137058 <https://reviews.llvm.org/D137058>, where there is no modules cache. I know it is a little bit frustrating to change the demo all the time... but let's turn for that direction now.

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

https://reviews.llvm.org/D134267