Skip to content

Commit 9c5173f

Browse files
committed
Add test to catch performance regressions
1 parent 0859e5e commit 9c5173f

File tree

2 files changed

+26
-1
lines changed

2 files changed

+26
-1
lines changed

src/librustc_trans/trans/intrinsic.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -362,7 +362,7 @@ pub fn trans_intrinsic_call<'a, 'blk, 'tcx>(mut bcx: Block<'blk, 'tcx>,
362362
(_, "init") => {
363363
let tp_ty = *substs.types.get(FnSpace, 0);
364364
if !return_type_is_void(ccx, tp_ty) {
365-
// Just zero out the stack slot
365+
// Just zero out the stack slot. (See comment on base::memzero for explaination)
366366
zero_mem(bcx, llresult, tp_ty);
367367
}
368368
C_nil(ccx)

src/test/run-pass/init-large-type.rs

Lines changed: 25 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,25 @@
1+
// Copyright 2015 The Rust Project Developers. See the COPYRIGHT
2+
// file at the top-level directory of this distribution and at
3+
// http://rust-lang.org/COPYRIGHT.
4+
//
5+
// Licensed under the Apache License, Version 2.0 <LICENSE-APACHE or
6+
// http://www.apache.org/licenses/LICENSE-2.0> or the MIT license
7+
// <LICENSE-MIT or http://opensource.org/licenses/MIT>, at your
8+
// option. This file may not be copied, modified, or distributed
9+
// except according to those terms.
10+
11+
// Makes sure that zero-initializing large types is reasonably fast,
12+
// Doing it incorrectly causes massive slowdown in LLVM during
13+
// optimisation.
14+
15+
#![feature(intrinsics)]
16+
17+
extern "rust-intrinsic" {
18+
pub fn init<T>() -> T;
19+
}
20+
21+
const SIZE: usize = 512 * 1024 * 1024;
22+
23+
fn main() {
24+
let _memory: [u8; SIZE] = unsafe { init() };
25+
}

0 commit comments

Comments
 (0)