[PATCH] D61452: [WebAssembly] Always include <sysroot>/lib in library path

Sam Clegg via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu May 2 13:35:33 PDT 2019


sbc100 added a comment.

Even in the case of wasm64 it might well be that somebody wants to build two different sysroots (as they would be arm and arm64 on linux).   Even if we don't choose to ship a single arch sysroot for wasi somebody else might.

This doesn't seem like it does any harm to me.  It arguable I suppose that it allowing library authors to delay their transition to mulity-arch install.  I don't see why we (WebAssembly driver) should be forcing them to do that.   This just allows more flexibility for downstream consumers of the compiler.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D61452





More information about the cfe-commits mailing list