thakis reopened this revision.
thakis added a comment.
This revision is now accepted and ready to land.

Please don't add code to the driver that runs programs off PATH. Nothing else 
does this. If you need to run an external program, look for it next to the 
compiler, like we do for gas with -fno-integrated-as, linker, etc. People can 
use the existing -B flag to make clang look elsewhere if they need that.

It's also somewhat uncommon to run non-LLVM binaries, but I can see the need 
for doing that at least during some transition period (like we did with the 
assembler).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70500



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to