OpenE2K/rust
2
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Remove incorrect assert in trans

Browse Source 
As an optimization, once unboxed closures receive their environment by
value if it fits within the size of an `int`.  An assert in this code
path assumed that this would only occur if the environment had no more
than a single free variable in it, but multiple smaller free variables
can easily be packed into the space of an `int`, particularly if any
of them are 0-sized.  The assert can simply be removed.

Closes #18652
This commit is contained in:
Brian Koropoff 2014-11-06 20:40:32 -08:00
parent b03a275519
commit c3c72239b7
1 changed files with 0 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
src/librustc/middle/trans/closure.rs
View File

@ -322,7 +322,6 @@ fn load_unboxed_closure_environment<'blk, 'tcx>(
self_type, self_type,
"unboxed_closure_env"); "unboxed_closure_env");
store_ty(bcx, bcx.fcx.llenv.unwrap(), datum.val, self_type); store_ty(bcx, bcx.fcx.llenv.unwrap(), datum.val, self_type);
assert!(freevars.len() <= 1);
datum.val datum.val
} else { } else {
bcx.fcx.llenv.unwrap() bcx.fcx.llenv.unwrap()