diff options
| author | bors <bors@rust-lang.org> | 2022-01-01 13:28:13 +0000 |
|---|---|---|
| committer | bors <bors@rust-lang.org> | 2022-01-01 13:28:13 +0000 |
| commit | da4a39264beea6305720100448cccd78c4eba737 (patch) | |
| tree | 939fe89699cce79804e4a397b098f9e9ee6d4315 /src/test/codegen/src-hash-algorithm/src-hash-algorithm-sha256.rs | |
| parent | e6e46055df8707e4780510f5f3794b4fbee07203 (diff) | |
| parent | f3e1f33e3c725da999fc9b9b45ed5008a87a0149 (diff) | |
| download | rust-da4a39264beea6305720100448cccd78c4eba737.tar.gz rust-da4a39264beea6305720100448cccd78c4eba737.zip | |
Auto merge of #92419 - erikdesjardins:coldland, r=nagisa
Mark drop calls in landing pads `cold` instead of `noinline` Now that deferred inlining has been disabled in LLVM (#92110), this shouldn't cause catastrophic size blowup. I confirmed that the test cases from https://github.com/rust-lang/rust/issues/41696#issuecomment-298696944 still compile quickly (<1s) after this change. ~Although note that I wasn't able to reproduce the original issue using a recent rustc/llvm with deferred inlining enabled, so those tests may no longer be representative. I was also unable to create a modified test case that reproduced the original issue.~ (edit: I reproduced it on CI by accident--the first commit timed out on the LLVM 12 builder, because I forgot to make it conditional on LLVM version) r? `@nagisa` cc `@arielb1` (this effectively reverts #42771 "mark calls in the unwind path as !noinline") cc `@RalfJung` (fixes #46515) edit: also fixes #87055
Diffstat (limited to 'src/test/codegen/src-hash-algorithm/src-hash-algorithm-sha256.rs')
0 files changed, 0 insertions, 0 deletions