Skip to content

Commit

Permalink
replace RetVoid with Unreachable to fix lint
Browse files Browse the repository at this point in the history
The function is marked `noreturn`, so it shouldn't return.
  • Loading branch information
thestinger committed Nov 3, 2013
1 parent a248e34 commit ddbd89b
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/librustc/middle/trans/intrinsic.rs
Original file line number Diff line number Diff line change
Expand Up @@ -221,7 +221,7 @@ pub fn trans_intrinsic(ccx: @mut CrateContext,
"abort" => {
let llfn = bcx.ccx().intrinsics.get_copy(&("llvm.trap"));
Call(bcx, llfn, [], []);
RetVoid(bcx);
Unreachable(bcx);
}
"size_of" => {
let tp_ty = substs.tys[0];
Expand Down

5 comments on commit ddbd89b

@bors
Copy link
Contributor

@bors bors commented on ddbd89b Nov 3, 2013

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bors
Copy link
Contributor

@bors bors commented on ddbd89b Nov 3, 2013

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

merging thestinger/rust/abort = ddbd89b into auto

@bors
Copy link
Contributor

@bors bors commented on ddbd89b Nov 3, 2013

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thestinger/rust/abort = ddbd89b merged ok, testing candidate = 318e1da

@bors
Copy link
Contributor

@bors bors commented on ddbd89b Nov 3, 2013

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bors
Copy link
Contributor

@bors bors commented on ddbd89b Nov 3, 2013

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fast-forwarding master to auto = 318e1da

Please sign in to comment.