[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