diff options
| author | Eduard-Mihai Burtescu <edy.burt@gmail.com> | 2017-12-26 03:20:35 +0200 |
|---|---|---|
| committer | Eduard-Mihai Burtescu <edy.burt@gmail.com> | 2017-12-26 22:35:53 +0200 |
| commit | 57bb8ab832a1bfc732d06bace1fc55c19677d9fa (patch) | |
| tree | 926b82fe0970ffdbfa32033b6fbf6e8ff6c6906b /src | |
| parent | 269827ced91bb2b702d4cb62e3e164b225f73157 (diff) | |
| download | rust-57bb8ab832a1bfc732d06bace1fc55c19677d9fa.tar.gz rust-57bb8ab832a1bfc732d06bace1fc55c19677d9fa.zip | |
rustc_trans: support ZST indexing involving uninhabited types.
Diffstat (limited to 'src')
| -rw-r--r-- | src/librustc_trans/mir/operand.rs | 19 | ||||
| -rw-r--r-- | src/test/run-pass/issue-46855.rs | 34 |
2 files changed, 50 insertions, 3 deletions
diff --git a/src/librustc_trans/mir/operand.rs b/src/librustc_trans/mir/operand.rs index c88235371f9..05af48761a1 100644 --- a/src/librustc_trans/mir/operand.rs +++ b/src/librustc_trans/mir/operand.rs @@ -267,9 +267,22 @@ impl<'a, 'tcx> MirContext<'a, 'tcx> { // Moves out of scalar and scalar pair fields are trivial. if let &mir::Place::Projection(ref proj) = place { - if let mir::ProjectionElem::Field(ref f, _) = proj.elem { - if let Some(o) = self.maybe_trans_consume_direct(bcx, &proj.base) { - return Some(o.extract_field(bcx, f.index())); + if let Some(o) = self.maybe_trans_consume_direct(bcx, &proj.base) { + match proj.elem { + mir::ProjectionElem::Field(ref f, _) => { + return Some(o.extract_field(bcx, f.index())); + } + mir::ProjectionElem::Index(_) | + mir::ProjectionElem::ConstantIndex { .. } => { + // ZSTs don't require any actual memory access. + // FIXME(eddyb) deduplicate this with the identical + // checks in `trans_consume` and `extract_field`. + let elem = o.layout.field(bcx.ccx, 0); + if elem.is_zst() { + return Some(OperandRef::new_zst(bcx.ccx, elem)); + } + } + _ => {} } } } diff --git a/src/test/run-pass/issue-46855.rs b/src/test/run-pass/issue-46855.rs new file mode 100644 index 00000000000..d864d55c939 --- /dev/null +++ b/src/test/run-pass/issue-46855.rs @@ -0,0 +1,34 @@ +// Copyright 2017 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. + +// compile-flags: -Zmir-opt-level=1 + +#![feature(slice_patterns)] + +use std::mem; + +#[derive(Copy, Clone)] +enum Never {} + +union Foo { + a: u64, + b: Never +} + +fn foo(xs: [(Never, u32); 1]) -> u32 { xs[0].1 } + +fn bar([(_, x)]: [(Never, u32); 1]) -> u32 { x } + +fn main() { + println!("{}", mem::size_of::<Foo>()); + + let f = [Foo { a: 42 }, Foo { a: 10 }]; + println!("{:?}", unsafe { f[0].a }); +} |
