Add separate impl of unwrap_failed to avoid constructing trait objects

diff --git a/library/core/src/result.rs b/library/core/src/result.rs
index e6b8c8ec3385a09077f2f743e975ec1d63d63558..ab067d57d082abc66ae74307382457573f88d874 100644 (file)
--- a/library/core/src/result.rs
+++ b/library/core/src/result.rs
@@ -1653,6 +1653,7 @@ pub const fn into_ok_or_err(self) -> T {
 }
 
 // This is a separate function to reduce the code size of the methods
+#[cfg(not(feature = "panic_immediate_abort"))]
 #[inline(never)]
 #[cold]
 #[track_caller]
@@ -1660,6 +1661,18 @@ fn unwrap_failed(msg: &str, error: &dyn fmt::Debug) -> ! {
     panic!("{}: {:?}", msg, error)
 }
 
+// This is a separate function to avoid constructing a `dyn Debug`
+// that gets immediately thrown away, since vtables don't get cleaned up
+// by dead code elimination if a trait object is constructed even if it goes
+// unused
+#[cfg(feature = "panic_immediate_abort")]
+#[inline]
+#[cold]
+#[track_caller]
+fn unwrap_failed<T>(_msg: &str, _error: &T) -> ! {
+    panic!()
+}
+
 /////////////////////////////////////////////////////////////////////////////
 // Trait implementations
 /////////////////////////////////////////////////////////////////////////////