From 039f0f4e7ef76482a41071408b8d627a9ab0e061 Mon Sep 17 00:00:00 2001 From: Peter Marheine Date: Mon, 12 Oct 2015 16:23:47 -0600 Subject: [PATCH 1/2] win64/msvc: large or oddly-sized types pass by-ref --- src/librustc_trans/trans/cabi_x86_win64.rs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/librustc_trans/trans/cabi_x86_win64.rs b/src/librustc_trans/trans/cabi_x86_win64.rs index 0a39150dbd36..120c8dc0384c 100644 --- a/src/librustc_trans/trans/cabi_x86_win64.rs +++ b/src/librustc_trans/trans/cabi_x86_win64.rs @@ -46,7 +46,7 @@ pub fn compute_abi_info(ccx: &CrateContext, 2 => ArgType::direct(t, Some(Type::i16(ccx)), None, None), 4 => ArgType::direct(t, Some(Type::i32(ccx)), None, None), 8 => ArgType::direct(t, Some(Type::i64(ccx)), None, None), - _ => ArgType::indirect(t, Some(Attribute::ByVal)) + _ => ArgType::indirect(t, None) } } _ => { From 95721d3b136c1459f309cd107e43c4edc474c052 Mon Sep 17 00:00:00 2001 From: Peter Marheine Date: Wed, 14 Oct 2015 14:25:38 -0600 Subject: [PATCH 2/2] Add test case for #28676. --- src/rt/rust_test_helpers.c | 4 ++++ src/test/run-pass/issue-28676.rs | 40 ++++++++++++++++++++++++++++++++ 2 files changed, 44 insertions(+) create mode 100644 src/test/run-pass/issue-28676.rs diff --git a/src/rt/rust_test_helpers.c b/src/rt/rust_test_helpers.c index 8824cef2a816..f7895d694c82 100644 --- a/src/rt/rust_test_helpers.c +++ b/src/rt/rust_test_helpers.c @@ -218,3 +218,7 @@ uint64_t get_y(struct S s) { uint64_t get_z(struct S s) { return s.z; } + +uint64_t get_c_many_params(void *a, void *b, void *c, void *d, struct quad f) { + return f.c; +} diff --git a/src/test/run-pass/issue-28676.rs b/src/test/run-pass/issue-28676.rs new file mode 100644 index 000000000000..b8d43c392dd9 --- /dev/null +++ b/src/test/run-pass/issue-28676.rs @@ -0,0 +1,40 @@ +// Copyright 2012-2015 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 or the MIT license +// , at your +// option. This file may not be copied, modified, or distributed +// except according to those terms. +// + +#[derive(Copy, Clone)] +pub struct Quad { a: u64, b: u64, c: u64, d: u64 } + +mod rustrt { + use super::Quad; + + #[link(name = "rust_test_helpers")] + extern { + pub fn get_c_many_params(_: *const (), _: *const (), + _: *const (), _: *const (), f: Quad) -> u64; + } +} + +fn test() { + unsafe { + let null = std::ptr::null(); + let q = Quad { + a: 1, + b: 2, + c: 3, + d: 4 + }; + assert_eq!(rustrt::get_c_many_params(null, null, null, null, q), q.c); + } +} + +pub fn main() { + test(); +}