2016-02-29 02:36:13 -06:00
|
|
|
//! lint on C-like enums that are `repr(isize/usize)` and have values that don't fit into an `i32`
|
|
|
|
|
|
|
|
use rustc::lint::*;
|
2016-03-31 10:05:43 -05:00
|
|
|
use rustc::middle::const_val::ConstVal;
|
|
|
|
use rustc_const_math::*;
|
2016-04-07 10:46:48 -05:00
|
|
|
use rustc::hir::*;
|
2016-02-29 02:36:13 -06:00
|
|
|
use utils::span_lint;
|
|
|
|
|
2016-08-06 02:55:04 -05:00
|
|
|
/// **What it does:** Checks for C-like enumerations that are
|
|
|
|
/// `repr(isize/usize)` and have values that don't fit into an `i32`.
|
2016-02-29 02:36:13 -06:00
|
|
|
///
|
2016-08-06 02:55:04 -05:00
|
|
|
/// **Why is this bad?** This will truncate the variant value on 32 bit
|
|
|
|
/// architectures, but works fine on 64 bit.
|
2016-02-29 02:36:13 -06:00
|
|
|
///
|
2016-08-06 02:55:04 -05:00
|
|
|
/// **Known problems:** None.
|
2016-02-29 02:36:13 -06:00
|
|
|
///
|
2016-07-15 17:25:44 -05:00
|
|
|
/// **Example:**
|
|
|
|
/// ```rust
|
|
|
|
/// #[repr(usize)]
|
|
|
|
/// enum NonPortable {
|
|
|
|
/// X = 0x1_0000_0000,
|
|
|
|
/// Y = 0
|
|
|
|
/// }
|
|
|
|
/// ```
|
2016-02-29 02:36:13 -06:00
|
|
|
declare_lint! {
|
2016-08-06 03:18:36 -05:00
|
|
|
pub ENUM_CLIKE_UNPORTABLE_VARIANT,
|
|
|
|
Warn,
|
|
|
|
"C-like enums that are `repr(isize/usize)` and have values that don't fit into an `i32`"
|
2016-02-29 02:36:13 -06:00
|
|
|
}
|
|
|
|
|
2016-06-10 09:17:20 -05:00
|
|
|
pub struct UnportableVariant;
|
2016-02-29 02:36:13 -06:00
|
|
|
|
2016-06-10 09:17:20 -05:00
|
|
|
impl LintPass for UnportableVariant {
|
2016-02-29 02:36:13 -06:00
|
|
|
fn get_lints(&self) -> LintArray {
|
|
|
|
lint_array!(ENUM_CLIKE_UNPORTABLE_VARIANT)
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2016-12-07 06:13:40 -06:00
|
|
|
impl<'a, 'tcx> LateLintPass<'a, 'tcx> for UnportableVariant {
|
2016-02-29 02:36:13 -06:00
|
|
|
#[allow(cast_possible_truncation, cast_sign_loss)]
|
2016-12-07 06:13:40 -06:00
|
|
|
fn check_item(&mut self, cx: &LateContext<'a, 'tcx>, item: &'tcx Item) {
|
2016-02-29 02:36:13 -06:00
|
|
|
if let ItemEnum(ref def, _) = item.node {
|
|
|
|
for var in &def.variants {
|
|
|
|
let variant = &var.node;
|
2017-01-04 16:18:11 -06:00
|
|
|
if let Some(body_id) = variant.disr_expr {
|
2016-03-15 14:09:53 -05:00
|
|
|
use rustc_const_eval::*;
|
2017-04-19 05:17:12 -05:00
|
|
|
let constcx = ConstContext::with_tables(cx.tcx, cx.tcx.body_tables(body_id));
|
2017-03-01 06:24:19 -06:00
|
|
|
let bad = match constcx.eval(&cx.tcx.hir.body(body_id).value) {
|
2016-03-15 14:09:53 -05:00
|
|
|
Ok(ConstVal::Integral(Usize(Us64(i)))) => i as u32 as u64 != i,
|
|
|
|
Ok(ConstVal::Integral(Isize(Is64(i)))) => i as i32 as i64 != i,
|
2016-02-29 02:36:13 -06:00
|
|
|
_ => false,
|
|
|
|
};
|
|
|
|
if bad {
|
|
|
|
span_lint(cx,
|
|
|
|
ENUM_CLIKE_UNPORTABLE_VARIANT,
|
|
|
|
var.span,
|
|
|
|
"Clike enum variant discriminant is not portable to 32-bit targets");
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|