diff options
| author | bors <bors@rust-lang.org> | 2018-05-18 19:36:26 +0000 |
|---|---|---|
| committer | bors <bors@rust-lang.org> | 2018-05-18 19:36:26 +0000 |
| commit | 952f344cdc0bca58d9f6c54dcfbae0890246e886 (patch) | |
| tree | 0eb2b5a5c90a4750faded1ccaf044cb45e13d17e | |
| parent | a722296b6ec17fecd3f16a7d3f9232b83e5de800 (diff) | |
| parent | 8b24644c42c0b6a0a3a3e561af2049001e82c9a4 (diff) | |
| download | rust-952f344cdc0bca58d9f6c54dcfbae0890246e886.tar.gz rust-952f344cdc0bca58d9f6c54dcfbae0890246e886.zip | |
Auto merge of #50697 - KiChjang:issue-50461, r=pnkfelix
Use EverInit instead of MaybeInit to determine initialization Fixes #50461. Fixes #50463.
| -rw-r--r-- | src/librustc_mir/borrow_check/mod.rs | 9 | ||||
| -rw-r--r-- | src/test/run-pass/nll/issue-50461-used-mut-from-moves.rs | 25 |
2 files changed, 31 insertions, 3 deletions
diff --git a/src/librustc_mir/borrow_check/mod.rs b/src/librustc_mir/borrow_check/mod.rs index 3d39042e9f4..5d6d4619c5e 100644 --- a/src/librustc_mir/borrow_check/mod.rs +++ b/src/librustc_mir/borrow_check/mod.rs @@ -1808,9 +1808,9 @@ impl<'cx, 'gcx, 'tcx> MirBorrowckCtxt<'cx, 'gcx, 'tcx> { } } Reservation(WriteKind::Move) + | Write(WriteKind::Move) | Reservation(WriteKind::StorageDeadOrDrop) | Reservation(WriteKind::MutableBorrow(BorrowKind::Shared)) - | Write(WriteKind::Move) | Write(WriteKind::StorageDeadOrDrop) | Write(WriteKind::MutableBorrow(BorrowKind::Shared)) => { if let Err(_place_err) = self.is_mutable(place, is_local_mutation_allowed) { @@ -1849,8 +1849,11 @@ impl<'cx, 'gcx, 'tcx> MirBorrowckCtxt<'cx, 'gcx, 'tcx> { // mutated, then it is justified to be annotated with the `mut` // keyword, since the mutation may be a possible reassignment. let mpi = self.move_data.rev_lookup.find_local(*local); - if flow_state.inits.contains(&mpi) { - self.used_mut.insert(*local); + let ii = &self.move_data.init_path_map[mpi]; + for index in ii { + if flow_state.ever_inits.contains(index) { + self.used_mut.insert(*local); + } } } } diff --git a/src/test/run-pass/nll/issue-50461-used-mut-from-moves.rs b/src/test/run-pass/nll/issue-50461-used-mut-from-moves.rs new file mode 100644 index 00000000000..d5cf122bf3b --- /dev/null +++ b/src/test/run-pass/nll/issue-50461-used-mut-from-moves.rs @@ -0,0 +1,25 @@ +// Copyright 2016 The Rust Project Developers. See the COPYRIGHT +// file at the top-level directory of this distribution and at +// http://rust-lang.org/COPYRIGHT. +// +// Licensed under the Apache License, Version 2.0 <LICENSE-APACHE or +// http://www.apache.org/licenses/LICENSE-2.0> or the MIT license +// <LICENSE-MIT or http://opensource.org/licenses/MIT>, at your +// option. This file may not be copied, modified, or distributed +// except according to those terms. + +#![feature(nll)] +#![deny(unused_mut)] + +struct Foo { + pub value: i32 +} + +fn use_foo_mut(mut foo: Foo) { + foo = foo; + println!("{}", foo.value); +} + +fn main() { + use_foo_mut(Foo { value: 413 }); +} |
