Skip to content

Commit 07dd9fc

Browse files
authored
Rollup merge of rust-lang#69825 - lcnr:discriminant, r=oli-obk
make `mem::discriminant` const implements rust-lang#69821, which could be used as a tracking issue for `const_discriminant`. Should this be added to the meta tracking issue rust-lang#57563? @Lokathor
2 parents deb6efc + 7b3e3ff commit 07dd9fc

File tree

6 files changed

+50
-1
lines changed

6 files changed

+50
-1
lines changed

src/libcore/intrinsics.rs

+1
Original file line numberDiff line numberDiff line change
@@ -1862,6 +1862,7 @@ extern "rust-intrinsic" {
18621862
///
18631863
/// The stabilized version of this intrinsic is
18641864
/// [`std::mem::discriminant`](../../std/mem/fn.discriminant.html)
1865+
#[rustc_const_unstable(feature = "const_discriminant", issue = "69821")]
18651866
pub fn discriminant_value<T>(v: &T) -> u64;
18661867

18671868
/// Rust's "try catch" construct which invokes the function pointer `f` with

src/libcore/lib.rs

+1
Original file line numberDiff line numberDiff line change
@@ -72,6 +72,7 @@
7272
#![feature(concat_idents)]
7373
#![feature(const_ascii_ctype_on_intrinsics)]
7474
#![feature(const_alloc_layout)]
75+
#![feature(const_discriminant)]
7576
#![feature(const_if_match)]
7677
#![feature(const_loop)]
7778
#![feature(const_checked_int_methods)]

src/libcore/mem/mod.rs

+2-1
Original file line numberDiff line numberDiff line change
@@ -870,6 +870,7 @@ impl<T> fmt::Debug for Discriminant<T> {
870870
/// assert_ne!(mem::discriminant(&Foo::B(3)), mem::discriminant(&Foo::C(3)));
871871
/// ```
872872
#[stable(feature = "discriminant_value", since = "1.21.0")]
873-
pub fn discriminant<T>(v: &T) -> Discriminant<T> {
873+
#[rustc_const_unstable(feature = "const_discriminant", issue = "69821")]
874+
pub const fn discriminant<T>(v: &T) -> Discriminant<T> {
874875
Discriminant(intrinsics::discriminant_value(v), PhantomData)
875876
}

src/librustc_mir/interpret/intrinsics.rs

+5
Original file line numberDiff line numberDiff line change
@@ -216,6 +216,11 @@ impl<'mir, 'tcx, M: Machine<'mir, 'tcx>> InterpCx<'mir, 'tcx, M> {
216216
};
217217
self.write_scalar(val, dest)?;
218218
}
219+
sym::discriminant_value => {
220+
let place = self.deref_operand(args[0])?;
221+
let discr_val = self.read_discriminant(place.into())?.0;
222+
self.write_scalar(Scalar::from_uint(discr_val, dest.layout.size), dest)?;
223+
}
219224
sym::unchecked_shl
220225
| sym::unchecked_shr
221226
| sym::unchecked_add

src/librustc_span/symbol.rs

+1
Original file line numberDiff line numberDiff line change
@@ -265,6 +265,7 @@ symbols! {
265265
derive,
266266
diagnostic,
267267
direct,
268+
discriminant_value,
268269
doc,
269270
doc_alias,
270271
doc_cfg,
+40
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,40 @@
1+
// run-pass
2+
#![feature(const_discriminant)]
3+
#![allow(dead_code)]
4+
5+
use std::mem::{discriminant, Discriminant};
6+
7+
// `discriminant(const_expr)` may get const-propagated.
8+
// As we want to check that const-eval is equal to ordinary exection,
9+
// we wrap `const_expr` with a function which is not const to prevent this.
10+
#[inline(never)]
11+
fn identity<T>(x: T) -> T { x }
12+
13+
enum Test {
14+
A(u8),
15+
B,
16+
C { a: u8, b: u8 },
17+
}
18+
19+
const TEST_A: Discriminant<Test> = discriminant(&Test::A(5));
20+
const TEST_A_OTHER: Discriminant<Test> = discriminant(&Test::A(17));
21+
const TEST_B: Discriminant<Test> = discriminant(&Test::B);
22+
23+
enum Void {}
24+
25+
enum SingleVariant {
26+
V,
27+
Never(Void),
28+
}
29+
30+
const TEST_V: Discriminant<SingleVariant> = discriminant(&SingleVariant::V);
31+
32+
fn main() {
33+
assert_eq!(TEST_A, TEST_A_OTHER);
34+
assert_eq!(TEST_A, discriminant(identity(&Test::A(17))));
35+
assert_eq!(TEST_B, discriminant(identity(&Test::B)));
36+
assert_ne!(TEST_A, TEST_B);
37+
assert_ne!(TEST_B, discriminant(identity(&Test::C { a: 42, b: 7 })));
38+
39+
assert_eq!(TEST_V, discriminant(identity(&SingleVariant::V)));
40+
}

0 commit comments

Comments
 (0)