Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Inline Finallyalizer::drop, allowing LLVM to optimize finally. #10918

Merged
merged 1 commit into from
Dec 14, 2013

Conversation

eddyb
Copy link
Member

@eddyb eddyb commented Dec 11, 2013

@eddyb
Copy link
Member Author

eddyb commented Dec 11, 2013

cc @cmr @huonw

@eddyb
Copy link
Member Author

eddyb commented Dec 11, 2013

r?

/* Reference result (TESTNAME=zero_1kb):
test vec::bench::zero_1kb_fixed_repeat                          ... bench:       240 ns/iter (+/- 16)
test vec::bench::zero_1kb_from_elem                             ... bench:       237 ns/iter (+/- 32)
test vec::bench::zero_1kb_loop_set                              ... bench:      1581 ns/iter (+/- 37)
test vec::bench::zero_1kb_mut_iter                              ... bench:       239 ns/iter (+/- 38)
test vec::bench::zero_1kb_set_memory                            ... bench:       236 ns/iter (+/- 9)
*/

* fixes the vec::from_elem regression caused by rust-lang#8780
* added 5 benchmarks for allocating a 1KB ~[u8] and zeroing it
bors added a commit that referenced this pull request Dec 13, 2013
* fixes the vec::from_elem regression caused by #8780
* added 5 benchmarks for allocating a 1KB `~[u8]` and zeroing it
* closes #7136
@bors bors closed this Dec 14, 2013
@bors bors merged commit 331c6ef into rust-lang:master Dec 14, 2013
@eddyb eddyb deleted the inline-finally-dtor branch December 14, 2013 07:27
flip1995 pushed a commit to flip1995/rust that referenced this pull request Jun 30, 2023
Fix `excessive_nesting` example

whoops 😅

changelog: none
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

vec::from_elem with primitives should be as fast as calloc
4 participants