ilya-biryukov added a comment.

In D70500#1758963 <https://reviews.llvm.org/D70500#1758963>, @sunfish wrote:

> If people feel using -B paths, and COMPILER_PATH, are appropriate, we can use 
> `GetProgramPath` itself, though note that that still does have a PATH 
> fallback.


I'm not an expert in driver code and how it should behave, but being consistent 
with how other tools are found definitely looks much better than special-casing 
a single tool.
Unless there are reasons why `wasm-opt` should be special, why not use the 
mechanism used by all other tools?


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