[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