mirror of
https://github.com/rust-lang/rust.git
synced 2024-10-30 22:12:15 +00:00
c2fd26a115
Currently, we assume that ScalarPair is always represented using a two-element struct, both as an immediate value and when stored in memory. This currently works fairly well, but runs into problems with https://github.com/rust-lang/rust/pull/116672, where a ScalarPair involving an i128 type can no longer be represented as a two-element struct in memory. For example, the tuple `(i32, i128)` needs to be represented in-memory as `{ i32, [3 x i32], i128 }` to satisfy alignment requirement. Using `{ i32, i128 }` instead will result in the second element being stored at the wrong offset (prior to LLVM 18). Resolve this issue by no longer requiring that the immediate and in-memory type for ScalarPair are the same. The in-memory type will now look the same as for normal struct types (and will include padding filler and similar), while the immediate type stays a simple two-element struct type. This also means that booleans in immediate ScalarPair are now represented as i1 rather than i8, just like we do everywhere else. The core change here is to llvm_type (which now treats ScalarPair as a normal struct) and immediate_llvm_type (which returns the two-element struct that llvm_type used to produce). The rest is fixing things up to no longer assume these are the same. In particular, this switches places that try to get pointers to the ScalarPair elements to use byte-geps instead of struct-geps.
36 lines
777 B
Rust
36 lines
777 B
Rust
// ignore-msvc
|
|
// ignore-wasm32-bare compiled with panic=abort by default
|
|
// needs-unwind
|
|
|
|
// compile-flags: -O -C no-prepopulate-passes
|
|
|
|
#![crate_type="lib"]
|
|
|
|
struct S;
|
|
|
|
impl Drop for S {
|
|
#[inline(never)]
|
|
fn drop(&mut self) {
|
|
}
|
|
}
|
|
|
|
#[inline(never)]
|
|
fn might_unwind() {
|
|
}
|
|
|
|
// CHECK-LABEL: @test
|
|
#[no_mangle]
|
|
pub fn test() {
|
|
let _s = S;
|
|
// Check that the personality slot alloca gets a lifetime start in each cleanup block, not just
|
|
// in the first one.
|
|
// CHECK: [[SLOT:%[0-9]+]] = alloca { ptr, i32, [1 x i32] }
|
|
// CHECK-LABEL: cleanup:
|
|
// CHECK: call void @llvm.lifetime.start.{{.*}}({{.*}})
|
|
// CHECK-LABEL: cleanup1:
|
|
// CHECK: call void @llvm.lifetime.start.{{.*}}({{.*}})
|
|
might_unwind();
|
|
let _t = S;
|
|
might_unwind();
|
|
}
|