Skip to content

Commit

Permalink
auto merge of #15226 : luqmana/rust/abol, r=pcwalton
Browse files Browse the repository at this point in the history
This was causing a ton of leaks in macro expansion.
  • Loading branch information
bors committed Jun 27, 2014
2 parents abdf71c + 04e64c0 commit afdfe40
Show file tree
Hide file tree
Showing 2 changed files with 44 additions and 2 deletions.
8 changes: 6 additions & 2 deletions src/librustc/middle/trans/expr.rs
Original file line number Diff line number Diff line change
Expand Up @@ -277,15 +277,19 @@ fn apply_adjustments<'a>(bcx: &'a Block<'a>,
auto_ref(bcx, datum, expr)
}

fn auto_borrow_obj<'a>(bcx: &'a Block<'a>,
fn auto_borrow_obj<'a>(mut bcx: &'a Block<'a>,
expr: &ast::Expr,
source_datum: Datum<Expr>)
-> DatumBlock<'a, Expr> {
let tcx = bcx.tcx();
let target_obj_ty = expr_ty_adjusted(bcx, expr);
debug!("auto_borrow_obj(target={})", target_obj_ty.repr(tcx));

let mut datum = source_datum.to_expr_datum();
// Arrange cleanup, if not already done. This is needed in
// case we are auto-borrowing a Box<Trait> to &Trait
let datum = unpack_datum!(
bcx, source_datum.to_lvalue_datum(bcx, "autoborrowobj", expr.id));
let mut datum = datum.to_expr_datum();
datum.ty = target_obj_ty;
DatumBlock::new(bcx, datum)
}
Expand Down
38 changes: 38 additions & 0 deletions src/test/run-pass/cleanup-auto-borrow-obj.rs
Original file line number Diff line number Diff line change
@@ -0,0 +1,38 @@
// Copyright 2014 The Rust Project Developers. See the COPYRIGHT
// file at the top-level directory of this distribution and at
// http://rust-lang.org/COPYRIGHT.
//
// Licensed under the Apache License, Version 2.0 <LICENSE-APACHE or
// http://www.apache.org/licenses/LICENSE-2.0> or the MIT license
// <LICENSE-MIT or http://opensource.org/licenses/MIT>, at your
// option. This file may not be copied, modified, or distributed
// except according to those terms.


// This would previously leak the Box<Trait> because we wouldn't
// schedule cleanups when auto borrowing trait objects.
// This program should be valgrind clean.


static mut DROP_RAN: bool = false;

struct Foo;
impl Drop for Foo {
fn drop(&mut self) {
unsafe { DROP_RAN = true; }
}
}


trait Trait {}
impl Trait for Foo {}

pub fn main() {
{
let _x: &Trait = box Foo as Box<Trait>;
}
unsafe {
assert!(DROP_RAN);
}
}

0 comments on commit afdfe40

Please sign in to comment.