[clang] [clang][modules][deps]Remove -F option from test for clang-scan-deps (PR #141614)
Cyndy Ishida via cfe-commits
cfe-commits at lists.llvm.org
Thu May 29 07:18:28 PDT 2025
cyndyishida wrote:
> I think that, in general, silently ignoring options that are not supported is a disservice to the user in that it misleads them into thinking that some functionality is there when it is not.
> -F is a darwin specific option in g++ so unless a target platform chooses to also support this option, it should be flagged as an error. I plan to introduce code in the future that will flag -F as an error on non-darwin targets for this reason.
@jamieschmeiser This is where I am confused. As far as I can tell, the clang support for respecting `-F` as a search path is not specific to darwin targets (I did a simple test using `-target powerpc64-unknown-linux-gnu` and a compilation that would fail without the search path option provided), so making it an error would remove preexisting functionality. And it's not obvious to me that it is valuable to remove such functionality to be more compatible with gcc.
https://github.com/llvm/llvm-project/pull/141614
More information about the cfe-commits
mailing list