From 6b84f7dd5ebcff6a43775a073c984f1b9ea13bcd Mon Sep 17 00:00:00 2001 From: Mark Simulacrum Date: Sun, 28 May 2017 08:12:19 -0600 Subject: [PATCH] Don't warn on lifetime generic no_mangle functions. --- src/librustc_lint/builtin.rs | 4 ++-- src/test/compile-fail/generic-no-mangle.rs | 10 ++++++++-- 2 files changed, 10 insertions(+), 4 deletions(-) diff --git a/src/librustc_lint/builtin.rs b/src/librustc_lint/builtin.rs index 39b8e568ab48d..106b6fea18eb9 100644 --- a/src/librustc_lint/builtin.rs +++ b/src/librustc_lint/builtin.rs @@ -1114,10 +1114,10 @@ impl<'a, 'tcx> LateLintPass<'a, 'tcx> for InvalidNoMangleItems { it.name); cx.span_lint(PRIVATE_NO_MANGLE_FNS, it.span, &msg); } - if generics.is_parameterized() { + if generics.is_type_parameterized() { cx.span_lint(NO_MANGLE_GENERIC_ITEMS, it.span, - "generic functions must be mangled"); + "functions generic over types must be mangled"); } } } diff --git a/src/test/compile-fail/generic-no-mangle.rs b/src/test/compile-fail/generic-no-mangle.rs index 2cb73cf2ef79b..2b4c94d281f73 100644 --- a/src/test/compile-fail/generic-no-mangle.rs +++ b/src/test/compile-fail/generic-no-mangle.rs @@ -11,9 +11,15 @@ #![deny(no_mangle_generic_items)] #[no_mangle] -pub fn foo() {} //~ ERROR generic functions must be mangled +pub fn foo() {} //~ ERROR functions generic over types must be mangled #[no_mangle] -pub extern fn bar() {} //~ ERROR generic functions must be mangled +pub extern fn bar() {} //~ ERROR functions generic over types must be mangled + +#[no_mangle] +pub fn baz(x: &i32) -> &i32 { x } + +#[no_mangle] +pub fn qux<'a>(x: &'a i32) -> &i32 { x } fn main() {}