[PATCH] D119850: Darwin: introduce a global override for debug prefix map entries

Adrian Prantl via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Feb 15 10:03:50 PST 2022


aprantl 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)
----------------
arphaman wrote:
> 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.
Good idea! Done.


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

https://reviews.llvm.org/D119850



More information about the cfe-commits mailing list