about summary refs log tree commit diff
path: root/src/rustllvm/RustWrapper.cpp
diff options
context:
space:
mode:
authorBrian Koropoff <bkoropoff@gmail.com>2014-09-11 22:50:40 -0700
committerBrian Koropoff <bkoropoff@gmail.com>2014-09-11 23:10:44 -0700
commit957229c21504c734c49a94f1685f82ac225df3e7 (patch)
treebcfe6fa933fcc7fdc5bf2a6b1efc55ea9d286b62 /src/rustllvm/RustWrapper.cpp
parentf9888ac33980aeb866fe2f904f2da436099136e5 (diff)
downloadrust-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