Skip to content

Commit a09f386

Browse files
committed
Auto merge of #32579 - michaelwoerister:stable-symbol-name-fix, r=eddyb
Fix typo in TxCtxt::crate_disambiguator() and add test case. r? @nikomatsakis Fixes #32554
2 parents 080edd9 + c2ede7e commit a09f386

File tree

4 files changed

+61
-1
lines changed

4 files changed

+61
-1
lines changed

src/librustc/ty/context.rs

+1-1
Original file line numberDiff line numberDiff line change
@@ -434,7 +434,7 @@ impl<'tcx> TyCtxt<'tcx> {
434434
if cnum == LOCAL_CRATE {
435435
self.sess.crate_disambiguator.get().as_str()
436436
} else {
437-
self.sess.cstore.crate_name(cnum)
437+
self.sess.cstore.crate_disambiguator(cnum)
438438
}
439439
}
440440

Original file line numberDiff line numberDiff line change
@@ -0,0 +1,18 @@
1+
-include ../tools.mk
2+
3+
# This test case makes sure that monomorphizations of the same function with the
4+
# same set of generic arguments will have the same symbol names when
5+
# instantiated in different crates.
6+
7+
dump-symbols = nm "$(TMPDIR)/lib$(1).rlib" \
8+
| grep "some_test_function" \
9+
| sed "s/^[0-9a-f]\{8,16\}/00000000/" \
10+
| sort \
11+
> "$(TMPDIR)/$(1).nm"
12+
13+
all:
14+
$(RUSTC) stable-symbol-names1.rs
15+
$(RUSTC) stable-symbol-names2.rs
16+
$(call dump-symbols,stable_symbol_names1)
17+
$(call dump-symbols,stable_symbol_names2)
18+
cmp "$(TMPDIR)/stable_symbol_names1.nm" "$(TMPDIR)/stable_symbol_names2.nm"
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,22 @@
1+
// Copyright 2016 The Rust Project Developers. See the COPYRIGHT
2+
// file at the top-level directory of this distribution and at
3+
// http://rust-lang.org/COPYRIGHT.
4+
//
5+
// Licensed under the Apache License, Version 2.0 <LICENSE-APACHE or
6+
// http://www.apache.org/licenses/LICENSE-2.0> or the MIT license
7+
// <LICENSE-MIT or http://opensource.org/licenses/MIT>, at your
8+
// option. This file may not be copied, modified, or distributed
9+
// except according to those terms.
10+
11+
#![crate_type="rlib"]
12+
13+
pub fn some_test_function<T>(t: T) -> T {
14+
t
15+
}
16+
17+
pub fn user() {
18+
some_test_function(0u32);
19+
some_test_function("abc");
20+
let x = 2u64;
21+
some_test_function(&x);
22+
}
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,20 @@
1+
// Copyright 2016 The Rust Project Developers. See the COPYRIGHT
2+
// file at the top-level directory of this distribution and at
3+
// http://rust-lang.org/COPYRIGHT.
4+
//
5+
// Licensed under the Apache License, Version 2.0 <LICENSE-APACHE or
6+
// http://www.apache.org/licenses/LICENSE-2.0> or the MIT license
7+
// <LICENSE-MIT or http://opensource.org/licenses/MIT>, at your
8+
// option. This file may not be copied, modified, or distributed
9+
// except according to those terms.
10+
11+
#![crate_type="rlib"]
12+
13+
extern crate stable_symbol_names1;
14+
15+
pub fn user() {
16+
stable_symbol_names1::some_test_function(1u32);
17+
stable_symbol_names1::some_test_function("def");
18+
let x = 2u64;
19+
stable_symbol_names1::some_test_function(&x);
20+
}

0 commit comments

Comments
 (0)