about summary refs log tree commit diff
path: root/src/rustllvm/RustWrapper.cpp
diff options
context:
space:
mode:
authorAlex Crichton <alex@alexcrichton.com>2015-01-30 12:02:51 -0800
committerAlex Crichton <alex@alexcrichton.com>2015-01-30 12:02:51 -0800
commitb446c5b67994bd9822e958e3e26122543086c1bc (patch)
tree316f8a04f62cf15b6ba69e68b628d8a381fa1ece /src/rustllvm/RustWrapper.cpp
parent9ff540ba3771a1c0a396ca3f7a685a94b4e0f622 (diff)
parentff25fd660ac7240f4c8379e74108a462a9aaba77 (diff)
downloadrust-b446c5b67994bd9822e958e3e26122543086c1bc.tar.gz
rust-b446c5b67994bd9822e958e3e26122543086c1bc.zip
rollup merge of #21495: richo/unexported-unmangled-lint
The usecase is that functions made visible to systems outside of the
rust ecosystem require the symbol to be visible.

This adds a lint for functions that are not exported, but also not mangled.

It has some gotchas:

[ ]: There is fallout in core that needs taking care of
[ ]: I'm not convinced the error message is correct
[ ]: It has no tests

~~However, there's an underlying issue which I'd like feedback on- which is that my belief that that non-pub functions would not have their symbols exported, however that seems not to be the case in the first case that this lint turned up in rustc (`rust_fail`), which intuition suggests has been working.~~

This seems to be a separate bug in rust, wherein the symbols are exported in binaries, but not in rlibs or dylibs. This lint would catch that case.
Diffstat (limited to 'src/rustllvm/RustWrapper.cpp')
0 files changed, 0 insertions, 0 deletions