[PATCH] D137058: [Driver] [Modules] Support -fsave-std-c++-module-file (1/2)

Chuanqi Xu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Nov 1 01:16:19 PDT 2022


ChuanqiXu added a comment.

In D137058#3896999 <https://reviews.llvm.org/D137058#3896999>, @dblaikie wrote:

> @ben.boeckel maybe this'd be the place to discuss the motivation for this feature (picking up from your comment here: https://reviews.llvm.org/D134267#3892629 )
>
> admittedly there is a flag for specifying the filename for Split DWARF too, but it's not a terribly official thing (doesn't have a '-f', '-g', etc prefix)

Although I agreed it would be better to wait for @ben.boeckel to answer the question, I still want to say "why we would offer something only if it is **terribly** bad". It looks like if it is normally bad, then we won't handle it and you need to workaround for it. I feel bad for the strategy.
Why can't we give some small help to make their lives easier.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D137058



More information about the cfe-commits mailing list