about summary refs log tree commit diff
path: root/src/rustllvm/RustWrapper.cpp
diff options
context:
space:
mode:
authorkennytm <kennytm@gmail.com>2018-01-31 16:36:15 +0800
committerGitHub <noreply@github.com>2018-01-31 16:36:15 +0800
commitfde67b6ae3baa7063224d5efd7fccbfbdfca383b (patch)
treed97e0debd215b51c01fd1d75b5518c491e0d1ce7 /src/rustllvm/RustWrapper.cpp
parent2c3e6ac887dd907f0a084021aa0472f9149c9132 (diff)
parenta2cc5d68a7d4a739b3da62540cb09f313de2f452 (diff)
downloadrust-fde67b6ae3baa7063224d5efd7fccbfbdfca383b.tar.gz
rust-fde67b6ae3baa7063224d5efd7fccbfbdfca383b.zip
Rollup merge of #47889 - alexcrichton:wasm-hidden-by-default, r=cramertj
rustc: Add an option to default hidden visibility

This commit adds a new option to target specifictions to specify that symbols
should be "hidden" visibility by default in LLVM. While there are no existing
targets that take advantage of this the `wasm32-unknown-unknown` target will
soon start to use this visibility. The LLD linker currently interprets `hidden`
as "don't export this from the wasm module" which is what we want for 90% of our
functions. While the LLD linker does have a "export this symbol" argument which
is what we use for other linkers, it was also somewhat easier to do this change
instead which'll involve less arguments flying around. Additionally there's no
need for non-`hidden` visibility for most of our symbols!

This change should not immediately impact the wasm targets as-is, but rather
this is laying the foundations for soon integrating LLD as a linker for wasm
code.
Diffstat (limited to 'src/rustllvm/RustWrapper.cpp')
0 files changed, 0 insertions, 0 deletions