diff options
| author | kennytm <kennytm@gmail.com> | 2018-05-23 00:26:08 +0800 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2018-05-23 00:26:08 +0800 |
| commit | eb92280287ff15e6683a7a32cd1c52f4a0493501 (patch) | |
| tree | f4c5708a514a96443dd8511860983fa45a58c05d /src/rustllvm/RustWrapper.cpp | |
| parent | 20298a93ab4a886960313d056932ac16a29901fb (diff) | |
| parent | 7ad9ef3c450e538be87a8be90bfcc5b6c9928791 (diff) | |
| download | rust-eb92280287ff15e6683a7a32cd1c52f4a0493501.tar.gz rust-eb92280287ff15e6683a7a32cd1c52f4a0493501.zip | |
Rollup merge of #50849 - est31:visit_closure_args, r=michaelwoerister
CheckLoopVisitor: also visit closure arguments
This turns the ICE #50581 in this code:
```rust
fn main() {
|_: [u8; break]| ();
}
```
from
```
'assertion failed: self.tcx.sess.err_count() > 0', librustc_typeck/check/mod.rs
```
to
```
librustc_mir/hair/cx/expr.rs:543: invalid loop id for break: not inside loop scope
```
which is an ICE as well but at a later stage during compilation and most importantly
fixes of bug #50576 will fix this as well.
As this "only" moves an ICE to a later stage, I didn't add any tests.
Now I have manually verified the default impls of the visitor trait to check whether we have missed any other opportunity to visit more stuff and coudln't find anything (except the missing `break` visit I've fixed in #50829 but that one was already r+'d so I didn't want to push more commits).
Diffstat (limited to 'src/rustllvm/RustWrapper.cpp')
0 files changed, 0 insertions, 0 deletions
