1
Fork 0

Rollup merge of #134073 - DianQK:fix-131227, r=oli-obk

dataflow_const_prop: do not eval a ptr address in SwitchInt

Fixes #131227.
This commit is contained in:
León Orell Valerian Liehr 2024-12-09 23:39:07 +01:00 committed by GitHub
commit cb5f03cbce
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
3 changed files with 26 additions and 18 deletions

View file

@ -534,8 +534,13 @@ impl<'a, 'tcx> ConstAnalysis<'a, 'tcx> {
// This allows the set of visited edges to grow monotonically with the lattice.
FlatSet::Bottom => TerminatorEdges::None,
FlatSet::Elem(scalar) => {
let choice = scalar.assert_scalar_int().to_bits_unchecked();
TerminatorEdges::Single(targets.target_for_value(choice))
if let Ok(scalar_int) = scalar.try_to_scalar_int() {
TerminatorEdges::Single(
targets.target_for_value(scalar_int.to_bits_unchecked()),
)
} else {
TerminatorEdges::SwitchInt { discr, targets }
}
}
FlatSet::Top => TerminatorEdges::SwitchInt { discr, targets },
}

View file

@ -1,16 +0,0 @@
//@ known-bug: #131227
//@ compile-flags: -Zmir-opt-level=3
static mut G: () = ();
fn myfunc() -> i32 {
let var = &raw mut G;
if var.is_null() {
return 0;
}
0
}
fn main() {
myfunc();
}

View file

@ -0,0 +1,19 @@
//! Issue: <https://github.com/rust-lang/rust/issues/131227>
//! Test that constant propagation in SwitchInt does not crash
//! when encountering a ptr-to-int transmute.
//@ check-pass
//@ compile-flags: -Zmir-enable-passes=+InstSimplify-before-inline,+DataflowConstProp
#![crate_type = "lib"]
static mut G: i32 = 0;
pub fn myfunc() -> i32 {
let var = &raw mut G;
let u: usize = unsafe { std::mem::transmute(var) };
match u {
0 => 0,
_ => 1,
}
}