Skip to content

Commit

Permalink
auto merge of #14643 : jakub-/rust/infinite-loop-unreachable, r=alexc…
Browse files Browse the repository at this point in the history
…richton
  • Loading branch information
bors committed Jun 5, 2014
2 parents a7ac6a4 + b9752b6 commit bb57e41
Show file tree
Hide file tree
Showing 3 changed files with 26 additions and 1 deletion.
2 changes: 1 addition & 1 deletion src/librustc/middle/trans/callee.rs
Original file line number Diff line number Diff line change
Expand Up @@ -802,7 +802,7 @@ pub fn trans_arg_datum<'a>(
// "undef" value, as such a value should never
// be inspected. It's important for the value
// to have type lldestty (the callee's expected type).
let llformal_arg_ty = type_of::type_of(ccx, formal_arg_ty);
let llformal_arg_ty = type_of::type_of_explicit_arg(ccx, formal_arg_ty);
unsafe {
val = llvm::LLVMGetUndef(llformal_arg_ty.to_ref());
}
Expand Down
4 changes: 4 additions & 0 deletions src/librustc/middle/trans/controlflow.rs
Original file line number Diff line number Diff line change
Expand Up @@ -264,6 +264,10 @@ pub fn trans_loop<'a>(bcx:&'a Block<'a>,

fcx.pop_loop_cleanup_scope(loop_id);

if ty::type_is_bot(node_id_type(bcx, loop_id)) {
Unreachable(next_bcx_in);
}

return next_bcx_in;
}

Expand Down
21 changes: 21 additions & 0 deletions src/test/run-pass/issue-13352.rs
Original file line number Diff line number Diff line change
@@ -0,0 +1,21 @@
// 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.

extern crate libc;

fn foo(_: proc()) {}

fn main() {
foo(loop {
unsafe { libc::exit(0 as libc::c_int); }
});
2u + (loop {});
-(loop {});
}

0 comments on commit bb57e41

Please sign in to comment.