]> git.lizzy.rs Git - rust.git/commit
Auto merge of #92419 - erikdesjardins:coldland, r=nagisa
authorbors <bors@rust-lang.org>
Sat, 1 Jan 2022 13:28:13 +0000 (13:28 +0000)
committerbors <bors@rust-lang.org>
Sat, 1 Jan 2022 13:28:13 +0000 (13:28 +0000)
commit4f49627c6fe2a32d1fed6310466bb0e1c535c0c0
treeb4b4ca1ad7daee4bd3721ec3d047e0a7da9df104
parent028c6f1454787c068ff5117e9000a1de4fd98374
parent64da730a41ac7df5fa8e03a82beb50abcdf5123e
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
compiler/rustc_codegen_gcc/src/builder.rs
compiler/rustc_codegen_llvm/src/llvm/ffi.rs
compiler/rustc_codegen_llvm/src/llvm_util.rs