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

Dan Gohman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu May 2 15:52:39 PDT 2019


sunfish added a comment.

If "$sysroot/lib" ends up coming to mean "wasm32" because people come to depend on that, then wasm64 may end up needing to be different in a gratuitous way, which I'd like to avoid.

I'd like to keep our sysroots tidy when we can. If some libraries are installed in `lib/wasm32-wasi` and others `lib` for no reason other than build script inertia, that's untidy.

It's not an absolute for me, but I am inclined to see if we can understand the need better first. Single-arch sysroots are possible either way, for example.


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