Skip to content
Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 6 additions & 6 deletions compiler/rustc_mir_transform/src/patch.rs
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,8 @@ pub(crate) struct MirPatch<'tcx> {
// Cached block for UnwindTerminate (with reason)
terminate_block: Option<(BasicBlock, UnwindTerminateReason)>,
body_span: Span,
/// The number of locals at the start of the transformation. New locals
/// get appended at the end.
next_local: usize,
/// The number of blocks at the start of the transformation. New blocks
/// get appended at the end.
Expand Down Expand Up @@ -176,8 +178,7 @@ impl<'tcx> MirPatch<'tcx> {
span: Span,
local_info: LocalInfo<'tcx>,
) -> Local {
let index = self.next_local;
self.next_local += 1;
let index = self.next_local + self.new_locals.len();
let mut new_decl = LocalDecl::new(ty, span);
**new_decl.local_info.as_mut().unwrap_crate_local() = local_info;
self.new_locals.push(new_decl);
Expand All @@ -186,17 +187,16 @@ impl<'tcx> MirPatch<'tcx> {

/// Queues the addition of a new temporary.
pub(crate) fn new_temp(&mut self, ty: Ty<'tcx>, span: Span) -> Local {
let index = self.next_local;
self.next_local += 1;
let index = self.next_local + self.new_locals.len();
self.new_locals.push(LocalDecl::new(ty, span));
Local::new(index)
}

/// Returns the type of a local that's newly-added in the patch.
pub(crate) fn local_ty(&self, local: Local) -> Ty<'tcx> {
let local = local.as_usize();
assert!(local < self.next_local);
let new_local_idx = self.new_locals.len() - (self.next_local - local);
assert!(local < self.next_local + self.new_locals.len());
let new_local_idx = local - self.next_local;
self.new_locals[new_local_idx].ty
}

Expand Down
5 changes: 4 additions & 1 deletion compiler/rustc_parse/src/parser/item.rs
Original file line number Diff line number Diff line change
Expand Up @@ -2664,7 +2664,10 @@ impl<'a> Parser<'a> {
// Rule out `unsafe extern {`.
&& !self.is_unsafe_foreign_mod()
// Rule out `async gen {` and `async gen move {`
&& !self.is_async_gen_block())
&& !self.is_async_gen_block()
// Rule out `const unsafe auto` and `const unsafe trait`.
&& !self.is_keyword_ahead(2, &[kw::Auto, kw::Trait])
)
})
// `extern ABI fn`
|| self.check_keyword_case(exp!(Extern), case)
Expand Down
21 changes: 17 additions & 4 deletions library/core/src/cell.rs
Original file line number Diff line number Diff line change
Expand Up @@ -252,7 +252,7 @@

use crate::cmp::Ordering;
use crate::fmt::{self, Debug, Display};
use crate::marker::{PhantomData, Unsize};
use crate::marker::{Destruct, PhantomData, Unsize};
use crate::mem::{self, ManuallyDrop};
use crate::ops::{self, CoerceUnsized, Deref, DerefMut, DerefPure, DispatchFromDyn};
use crate::panic::const_panic;
Expand Down Expand Up @@ -429,7 +429,11 @@ impl<T> Cell<T> {
/// ```
#[inline]
#[stable(feature = "rust1", since = "1.0.0")]
pub fn set(&self, val: T) {
#[rustc_const_unstable(feature = "const_cell_traits", issue = "147787")]
pub const fn set(&self, val: T)
where
T: [const] Destruct,
{
self.replace(val);
}

Expand Down Expand Up @@ -561,7 +565,12 @@ impl<T: Copy> Cell<T> {
/// ```
#[inline]
#[stable(feature = "cell_update", since = "1.88.0")]
pub fn update(&self, f: impl FnOnce(T) -> T) {
#[rustc_const_unstable(feature = "const_cell_traits", issue = "147787")]
pub const fn update(&self, f: impl [const] FnOnce(T) -> T)
where
// FIXME(const-hack): `Copy` should imply `const Destruct`
T: [const] Destruct,
{
let old = self.get();
self.set(f(old));
}
Expand Down Expand Up @@ -654,7 +663,11 @@ impl<T: Default> Cell<T> {
/// assert_eq!(c.into_inner(), 0);
/// ```
#[stable(feature = "move_cell", since = "1.17.0")]
pub fn take(&self) -> T {
#[rustc_const_unstable(feature = "const_cell_traits", issue = "147787")]
pub const fn take(&self) -> T
where
T: [const] Default,
{
self.replace(Default::default())
}
}
Expand Down
1 change: 1 addition & 0 deletions library/coretests/tests/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@
#![feature(char_internals)]
#![feature(char_max_len)]
#![feature(clone_to_uninit)]
#![feature(const_cell_traits)]
#![feature(const_cmp)]
#![feature(const_convert)]
#![feature(const_destruct)]
Expand Down
38 changes: 38 additions & 0 deletions library/coretests/tests/manually_drop.rs
Original file line number Diff line number Diff line change
Expand Up @@ -27,3 +27,41 @@ fn smoke() {
drop(x);
drop(y);
}

#[test]
fn const_drop_in_place() {
const COUNTER: usize = {
use core::cell::Cell;

let counter = Cell::new(0);

// only exists to make `Drop` indirect impl
#[allow(dead_code)]
struct Test<'a>(Dropped<'a>);

struct Dropped<'a>(&'a Cell<usize>);
impl const Drop for Dropped<'_> {
fn drop(&mut self) {
self.0.set(self.0.get() + 1);
}
}

let mut one = ManuallyDrop::new(Test(Dropped(&counter)));
let mut two = ManuallyDrop::new(Test(Dropped(&counter)));
let mut three = ManuallyDrop::new(Test(Dropped(&counter)));
assert!(counter.get() == 0);
unsafe {
ManuallyDrop::drop(&mut one);
}
assert!(counter.get() == 1);
unsafe {
ManuallyDrop::drop(&mut two);
}
assert!(counter.get() == 2);
unsafe {
ManuallyDrop::drop(&mut three);
}
counter.get()
};
assert_eq!(COUNTER, 3);
}
39 changes: 0 additions & 39 deletions library/coretests/tests/ptr.rs
Original file line number Diff line number Diff line change
Expand Up @@ -1045,42 +1045,3 @@ fn test_ptr_default() {
let default = PtrMutDefaultTest::default();
assert!(default.ptr.is_null());
}

#[test]
fn test_const_drop_in_place() {
const COUNTER: usize = {
let mut counter = 0;
let counter_ptr = &raw mut counter;

// only exists to make `Drop` indirect impl
#[allow(dead_code)]
struct Test(Dropped);

struct Dropped(*mut usize);
impl const Drop for Dropped {
fn drop(&mut self) {
unsafe {
*self.0 += 1;
}
}
}

let mut one = ManuallyDrop::new(Test(Dropped(counter_ptr)));
let mut two = ManuallyDrop::new(Test(Dropped(counter_ptr)));
let mut three = ManuallyDrop::new(Test(Dropped(counter_ptr)));
assert!(counter == 0);
unsafe {
ManuallyDrop::drop(&mut one);
}
assert!(counter == 1);
unsafe {
ManuallyDrop::drop(&mut two);
}
assert!(counter == 2);
unsafe {
ManuallyDrop::drop(&mut three);
}
counter
};
assert_eq!(COUNTER, 3);
}
13 changes: 13 additions & 0 deletions tests/ui/traits/const-traits/parse-const-unsafe-trait.rs
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@
// Test that `const unsafe trait` and `const unsafe auto trait` works.

//@ check-pass

#![feature(const_trait_impl)]
#![feature(auto_traits)]

pub const unsafe trait Owo {}
const unsafe trait OwO {}
pub const unsafe auto trait UwU {}
const unsafe auto trait Uwu {}

fn main() {}
Loading