[clang] [clang-scan-deps] Expand response files before the argument adjuster (PR #89950)
Alexandre Ganea via cfe-commits
cfe-commits at lists.llvm.org
Wed Apr 24 10:32:53 PDT 2024
aganea wrote:
The reason of this `/Fo` output path fiddling code is this: https://github.com/llvm/llvm-project/blob/main/clang/lib/Driver/ToolChains/Clang.cpp#L1072-L1082
It's because that highlighted code doesn't handle the clang-cl case, it doesn't consider `/Fo`. Probably because most people don't use `-MT`/`-MF` with clang-cl (it has to be escaped on the command-line such as `/clang:-MT /clang:-MF...`. I can fix it there (in Clang.cpp) instead, and remove all the related code in `clang_scan_deps_main()`.
https://github.com/llvm/llvm-project/pull/89950
More information about the cfe-commits
mailing list