diff options
| author | Esteban Küber <esteban@kuber.com.ar> | 2019-09-20 19:31:51 -0700 |
|---|---|---|
| committer | Esteban Küber <esteban@kuber.com.ar> | 2019-09-30 15:11:19 -0700 |
| commit | 2ae87ff59d1261d6295d82b0b29bc5440ada0b32 (patch) | |
| tree | bbc2306280e641024e653b85bd92ee087b6e2bc2 | |
| parent | 22bc9e1d9ca49ee4f5cd953088ab09c238a6dd26 (diff) | |
| download | rust-2ae87ff59d1261d6295d82b0b29bc5440ada0b32.tar.gz rust-2ae87ff59d1261d6295d82b0b29bc5440ada0b32.zip | |
Avoid ICE on return outside of fn with literal array
Do not ICE when encountering `enum E { A = return [0][0] }`.
| -rw-r--r-- | src/librustc_typeck/check/writeback.rs | 20 | ||||
| -rw-r--r-- | src/test/ui/issues/issue-64620.rs | 5 | ||||
| -rw-r--r-- | src/test/ui/issues/issue-64620.stderr | 9 |
3 files changed, 31 insertions, 3 deletions
diff --git a/src/librustc_typeck/check/writeback.rs b/src/librustc_typeck/check/writeback.rs index de78c1cdfab..0dbf4dd09bd 100644 --- a/src/librustc_typeck/check/writeback.rs +++ b/src/librustc_typeck/check/writeback.rs @@ -189,9 +189,23 @@ impl<'cx, 'tcx> WritebackCx<'cx, 'tcx> { if let hir::ExprKind::Index(ref base, ref index) = e.kind { let mut tables = self.fcx.tables.borrow_mut(); - // All valid indexing looks like this; might encounter non-valid indexes at this point - if let ty::Ref(_, base_ty, _) = tables.expr_ty_adjusted(&base).kind { - let index_ty = tables.expr_ty_adjusted(&index); + // All valid indexing looks like this; might encounter non-valid indexes at this point. + let base_ty = tables.expr_ty_adjusted_opt(&base).map(|t| &t.kind); + if base_ty.is_none() { + self.tcx().sess.delay_span_bug(e.span, &format!("bad base: `{:?}`", base)); + return; + } + if let Some(ty::Ref(_, base_ty, _)) = base_ty { + let index_ty = match tables.expr_ty_adjusted_opt(&index) { + Some(t) => t, + None => { + self.tcx().sess.delay_span_bug( + e.span, + &format!("bad index {:?} for base: `{:?}`", index, base), + ); + self.fcx.tcx.types.err + } + }; let index_ty = self.fcx.resolve_vars_if_possible(&index_ty); if base_ty.builtin_index().is_some() && index_ty == self.fcx.tcx.types.usize { diff --git a/src/test/ui/issues/issue-64620.rs b/src/test/ui/issues/issue-64620.rs new file mode 100644 index 00000000000..a62e5bf8d3c --- /dev/null +++ b/src/test/ui/issues/issue-64620.rs @@ -0,0 +1,5 @@ +enum Bug { + V1 = return [0][0] //~ERROR return statement outside of function body +} + +fn main() {} diff --git a/src/test/ui/issues/issue-64620.stderr b/src/test/ui/issues/issue-64620.stderr new file mode 100644 index 00000000000..f40ac4de32d --- /dev/null +++ b/src/test/ui/issues/issue-64620.stderr @@ -0,0 +1,9 @@ +error[E0572]: return statement outside of function body + --> $DIR/issue-64620.rs:2:10 + | +LL | V1 = return [0][0] + | ^^^^^^^^^^^^^ + +error: aborting due to previous error + +For more information about this error, try `rustc --explain E0572`. |
