diff options
| author | Alex Crichton <alex@alexcrichton.com> | 2015-01-30 12:02:51 -0800 |
|---|---|---|
| committer | Alex Crichton <alex@alexcrichton.com> | 2015-01-30 12:02:51 -0800 |
| commit | b446c5b67994bd9822e958e3e26122543086c1bc (patch) | |
| tree | 316f8a04f62cf15b6ba69e68b628d8a381fa1ece /src/rustllvm/RustWrapper.cpp | |
| parent | 9ff540ba3771a1c0a396ca3f7a685a94b4e0f622 (diff) | |
| parent | ff25fd660ac7240f4c8379e74108a462a9aaba77 (diff) | |
| download | rust-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
