Skip to content

Commit

Permalink
Rollup merge of #111587 - cbeuw:copy-for-deref, r=oli-obk
Browse files Browse the repository at this point in the history
Custom MIR: Support `Rvalue::CopyForDeref`

r? `@oli-obk` or `@tmiasko` or `@JakobDegen`
  • Loading branch information
matthiaskrgr authored May 15, 2023
2 parents 119b722 + 3d938dd commit eeebb65
Show file tree
Hide file tree
Showing 5 changed files with 37 additions and 12 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -154,6 +154,7 @@ impl<'tcx, 'body> ParseCtxt<'tcx, 'body> {
Ok(Rvalue::BinaryOp(BinOp::Offset, Box::new((ptr, offset))))
},
@call("mir_len", args) => Ok(Rvalue::Len(self.parse_place(args[0])?)),
@call("mir_copy_for_deref", args) => Ok(Rvalue::CopyForDeref(self.parse_place(args[0])?)),
ExprKind::Borrow { borrow_kind, arg } => Ok(
Rvalue::Ref(self.tcx.lifetimes.re_erased, *borrow_kind, self.parse_place(*arg)?)
),
Expand Down
3 changes: 2 additions & 1 deletion library/core/src/intrinsics/mir.rs
Original file line number Diff line number Diff line change
Expand Up @@ -228,7 +228,7 @@
//!
//! - Operands implicitly convert to `Use` rvalues.
//! - `&`, `&mut`, `addr_of!`, and `addr_of_mut!` all work to create their associated rvalue.
//! - [`Discriminant`] and [`Len`] have associated functions.
//! - [`Discriminant`], [`Len`], and [`CopyForDeref`] have associated functions.
//! - Unary and binary operations use their normal Rust syntax - `a * b`, `!c`, etc.
//! - The binary operation `Offset` can be created via [`Offset`].
//! - Checked binary operations are represented by wrapping the associated binop in [`Checked`].
Expand Down Expand Up @@ -279,6 +279,7 @@ define!("mir_storage_dead", fn StorageDead<T>(local: T));
define!("mir_deinit", fn Deinit<T>(place: T));
define!("mir_checked", fn Checked<T>(binop: T) -> (T, bool));
define!("mir_len", fn Len<T>(place: T) -> usize);
define!("mir_copy_for_deref", fn CopyForDeref<T>(place: T) -> T);
define!("mir_retag", fn Retag<T>(place: T));
define!("mir_move", fn Move<T>(place: T) -> T);
define!("mir_static", fn Static<T>(s: T) -> &'static T);
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@
// MIR for `copy_for_deref` after built

fn copy_for_deref(_1: (&i32, i32)) -> i32 {
let mut _0: i32; // return place in scope 0 at $DIR/projections.rs:+0:38: +0:41
let mut _2: &i32; // in scope 0 at $SRC_DIR/core/src/intrinsics/mir.rs:LL:COL

bb0: {
_2 = deref_copy (_1.0: &i32); // scope 0 at $DIR/projections.rs:+4:13: +4:37
_0 = (*_2); // scope 0 at $DIR/projections.rs:+5:13: +5:24
return; // scope 0 at $DIR/projections.rs:+6:13: +6:21
}
}
25 changes: 19 additions & 6 deletions tests/mir-opt/building/custom/projections.rs
Original file line number Diff line number Diff line change
Expand Up @@ -21,13 +21,10 @@ fn unions(u: U) -> i32 {
#[custom_mir(dialect = "analysis", phase = "post-cleanup")]
fn tuples(i: (u32, i32)) -> (u32, i32) {
mir!(
// FIXME(JakobDegen): This is necessary because we can't give type hints for `RET`
let temp: (u32, i32);
type RET = (u32, i32);
{
temp.0 = i.0;
temp.1 = i.1;

RET = temp;
RET.0 = i.0;
RET.1 = i.1;
Return()
}
)
Expand Down Expand Up @@ -71,6 +68,19 @@ fn simple_index(a: [i32; 10], b: &[i32]) -> i32 {
})
}

// EMIT_MIR projections.copy_for_deref.built.after.mir
#[custom_mir(dialect = "runtime", phase = "initial")]
fn copy_for_deref(x: (&i32, i32)) -> i32 {
mir!(
let temp: &i32;
{
temp = CopyForDeref(x.0);
RET = *temp;
Return()
}
)
}

fn main() {
assert_eq!(unions(U { a: 5 }), 5);
assert_eq!(tuples((5, 6)), (5, 6));
Expand All @@ -82,4 +92,7 @@ fn main() {
assert_eq!(o, Some(10));

assert_eq!(simple_index([0; 10], &[0; 10]), 0);

let one = 1;
assert_eq!(copy_for_deref((&one, one)), 1);
}
Original file line number Diff line number Diff line change
Expand Up @@ -2,12 +2,10 @@

fn tuples(_1: (u32, i32)) -> (u32, i32) {
let mut _0: (u32, i32); // return place in scope 0 at $DIR/projections.rs:+0:29: +0:39
let mut _2: (u32, i32); // in scope 0 at $SRC_DIR/core/src/intrinsics/mir.rs:LL:COL

bb0: {
(_2.0: u32) = (_1.0: u32); // scope 0 at $DIR/projections.rs:+5:13: +5:25
(_2.1: i32) = (_1.1: i32); // scope 0 at $DIR/projections.rs:+6:13: +6:25
_0 = _2; // scope 0 at $DIR/projections.rs:+8:13: +8:23
return; // scope 0 at $DIR/projections.rs:+9:13: +9:21
(_0.0: u32) = (_1.0: u32); // scope 0 at $DIR/projections.rs:+4:13: +4:24
(_0.1: i32) = (_1.1: i32); // scope 0 at $DIR/projections.rs:+5:13: +5:24
return; // scope 0 at $DIR/projections.rs:+6:13: +6:21
}
}

0 comments on commit eeebb65

Please sign in to comment.