[PATCH] D119850: Darwin: introduce a global override for debug prefix map entries
Alex Lorenz via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Feb 15 09:11:28 PST 2022
arphaman added inline comments.
================
Comment at: clang/lib/Driver/ToolChains/Clang.cpp:686
+ return;
+ if (!StringRef(GlobalRemapEntry).contains('='))
+ D.Diag(diag::err_drv_invalid_argument_to_option)
----------------
Suggestion: It might be good to factor out the code that verifies the value and either produces an error or adds an argument into some common helper as it's used right above this codepath as well.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D119850/new/
https://reviews.llvm.org/D119850
More information about the cfe-commits
mailing list