[PATCH] D70500: [WebAssembly] Enable use of wasm-opt and LTO-enabled system libraries

Wed Nov 27 06:18:05 PST 2019

sunfish added a comment.

In D70500#1759831 <https://reviews.llvm.org/D70500#1759831>, @ilya-biryukov wrote:

> 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?

@dschuff also suggested this approach offline, so it sounds like the way to go. I've now submitted https://reviews.llvm.org/D70780 which implements this.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70500