diff options
| author | bors <bors@rust-lang.org> | 2017-06-28 08:47:29 +0000 |
|---|---|---|
| committer | bors <bors@rust-lang.org> | 2017-06-28 08:47:29 +0000 |
| commit | 6b52a1162eff2469c0da88300577863e5cc0c78f (patch) | |
| tree | a76bffcb21a36e526d1e466b98c157bf06df0855 | |
| parent | 5bc89416388f0a852f6e48542bf2fc021682fcb5 (diff) | |
| parent | 71abfa7b913d79c7ff5a3cc32e2988d1aac86480 (diff) | |
| download | rust-6b52a1162eff2469c0da88300577863e5cc0c78f.tar.gz rust-6b52a1162eff2469c0da88300577863e5cc0c78f.zip | |
Auto merge of #42931 - arielb1:statement-visitor, r=eddyb
re-add the call to `super_statement` in EraseRegions The move gathering code is sensitive to type-equality - that is rather un-robust and I plan to fix it eventually, but that's a more invasive change. And we want to fix the visitor anyway. Fixes #42903. r? @eddyb
| -rw-r--r-- | src/librustc_mir/transform/erase_regions.rs | 5 | ||||
| -rw-r--r-- | src/test/run-pass/dynamic-drop.rs | 15 |
2 files changed, 18 insertions, 2 deletions
diff --git a/src/librustc_mir/transform/erase_regions.rs b/src/librustc_mir/transform/erase_regions.rs index e809695c180..eb283df869f 100644 --- a/src/librustc_mir/transform/erase_regions.rs +++ b/src/librustc_mir/transform/erase_regions.rs @@ -67,12 +67,13 @@ impl<'a, 'tcx> MutVisitor<'tcx> for EraseRegionsVisitor<'a, 'tcx> { } fn visit_statement(&mut self, - _block: BasicBlock, + block: BasicBlock, statement: &mut Statement<'tcx>, - _location: Location) { + location: Location) { if let StatementKind::EndRegion(_) = statement.kind { statement.kind = StatementKind::Nop; } + self.super_statement(block, statement, location); } } diff --git a/src/test/run-pass/dynamic-drop.rs b/src/test/run-pass/dynamic-drop.rs index 6725a0c547f..bdcd3eecc69 100644 --- a/src/test/run-pass/dynamic-drop.rs +++ b/src/test/run-pass/dynamic-drop.rs @@ -106,6 +106,18 @@ fn struct_dynamic_drop(a: &Allocator, c0: bool, c1: bool, c: bool) { } } +fn field_assignment(a: &Allocator, c0: bool) { + let mut x = (TwoPtrs(a.alloc(), a.alloc()), a.alloc()); + + x.1 = a.alloc(); + x.1 = a.alloc(); + + let f = (x.0).0; + if c0 { + (x.0).0 = f; + } +} + fn assignment2(a: &Allocator, c0: bool, c1: bool) { let mut _v = a.alloc(); let mut _w = a.alloc(); @@ -207,5 +219,8 @@ fn main() { run_test(|a| struct_dynamic_drop(a, true, true, false)); run_test(|a| struct_dynamic_drop(a, true, true, true)); + run_test(|a| field_assignment(a, false)); + run_test(|a| field_assignment(a, true)); + run_test_nopanic(|a| union1(a)); } |
