ignore item in thread_local!
This commit is contained in:
parent
844c06a7c7
commit
a9215d90c8
@ -6,6 +6,7 @@
|
|||||||
|
|
||||||
use clippy_utils::diagnostics::span_lint_and_then;
|
use clippy_utils::diagnostics::span_lint_and_then;
|
||||||
use clippy_utils::in_constant;
|
use clippy_utils::in_constant;
|
||||||
|
use clippy_utils::macros::macro_backtrace;
|
||||||
use if_chain::if_chain;
|
use if_chain::if_chain;
|
||||||
use rustc_hir::def::{DefKind, Res};
|
use rustc_hir::def::{DefKind, Res};
|
||||||
use rustc_hir::def_id::DefId;
|
use rustc_hir::def_id::DefId;
|
||||||
@ -17,7 +18,7 @@
|
|||||||
use rustc_middle::ty::adjustment::Adjust;
|
use rustc_middle::ty::adjustment::Adjust;
|
||||||
use rustc_middle::ty::{self, Const, Ty};
|
use rustc_middle::ty::{self, Const, Ty};
|
||||||
use rustc_session::{declare_lint_pass, declare_tool_lint};
|
use rustc_session::{declare_lint_pass, declare_tool_lint};
|
||||||
use rustc_span::{InnerSpan, Span, DUMMY_SP};
|
use rustc_span::{sym, InnerSpan, Span, DUMMY_SP};
|
||||||
use rustc_typeck::hir_ty_to_ty;
|
use rustc_typeck::hir_ty_to_ty;
|
||||||
|
|
||||||
// FIXME: this is a correctness problem but there's no suitable
|
// FIXME: this is a correctness problem but there's no suitable
|
||||||
@ -250,8 +251,14 @@ impl<'tcx> LateLintPass<'tcx> for NonCopyConst {
|
|||||||
fn check_item(&mut self, cx: &LateContext<'tcx>, it: &'tcx Item<'_>) {
|
fn check_item(&mut self, cx: &LateContext<'tcx>, it: &'tcx Item<'_>) {
|
||||||
if let ItemKind::Const(hir_ty, body_id) = it.kind {
|
if let ItemKind::Const(hir_ty, body_id) = it.kind {
|
||||||
let ty = hir_ty_to_ty(cx.tcx, hir_ty);
|
let ty = hir_ty_to_ty(cx.tcx, hir_ty);
|
||||||
|
if !macro_backtrace(it.span).last().map_or(false, |macro_call| {
|
||||||
if is_unfrozen(cx, ty) && is_value_unfrozen_poly(cx, body_id, ty) {
|
matches!(
|
||||||
|
cx.tcx.get_diagnostic_name(macro_call.def_id),
|
||||||
|
Some(sym::thread_local_macro)
|
||||||
|
)
|
||||||
|
}) && is_unfrozen(cx, ty)
|
||||||
|
&& is_value_unfrozen_poly(cx, body_id, ty)
|
||||||
|
{
|
||||||
lint(cx, Source::Item { item: it.span });
|
lint(cx, Source::Item { item: it.span });
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
@ -31,4 +31,9 @@ macro_rules! declare_const {
|
|||||||
static STATIC_TUPLE: (AtomicUsize, String) = (ATOMIC, STRING);
|
static STATIC_TUPLE: (AtomicUsize, String) = (ATOMIC, STRING);
|
||||||
//^ there should be no lints on this line
|
//^ there should be no lints on this line
|
||||||
|
|
||||||
|
// issue #8493
|
||||||
|
thread_local! {
|
||||||
|
static THREAD_LOCAL: Cell<i32> = const { Cell::new(0) };
|
||||||
|
}
|
||||||
|
|
||||||
fn main() {}
|
fn main() {}
|
||||||
|
Loading…
Reference in New Issue
Block a user