diff options
| author | Brian Koropoff <bkoropoff@gmail.com> | 2014-09-11 22:50:40 -0700 |
|---|---|---|
| committer | Brian Koropoff <bkoropoff@gmail.com> | 2014-09-11 23:10:44 -0700 |
| commit | 957229c21504c734c49a94f1685f82ac225df3e7 (patch) | |
| tree | bcfe6fa933fcc7fdc5bf2a6b1efc55ea9d286b62 /src/rustllvm/RustWrapper.cpp | |
| parent | f9888ac33980aeb866fe2f904f2da436099136e5 (diff) | |
| download | rust-957229c21504c734c49a94f1685f82ac225df3e7.tar.gz rust-957229c21504c734c49a94f1685f82ac225df3e7.zip | |
Fix check for existing crate when using --extern
When checking for an existing crate, compare against the `crate_metadata::name` field, which is the crate name which was requested during resolution, rather than the result of the `crate_metadata::name()` method, which is the crate name within the crate metadata, as these may not match when using the --extern option to `rustc`. This fixes spurious "multiple crate version" warnings under the following scenario: - The crate `foo`, is referenced multiple times - `--extern foo=./path/to/libbar.rlib` is specified to rustc - The internal crate name of `libbar.rlib` is not `foo` The behavior surrounding `Context::should_match_name` and the comments in `loader.rs` both lead me to believe that this scenario is intended to work. Fixes #17186
Diffstat (limited to 'src/rustllvm/RustWrapper.cpp')
0 files changed, 0 insertions, 0 deletions
