Always inline layout validation and pointer cast code to optimize out…

… panics in (#1997) (#2013) `validate_cast_and_convert_metadata`. If `try_cast_into` and `validate_cast_and_convert_metadata` are not inlined, then the compiler will include a panic in `validate_cast_and_convert_metadata`. If these functions are inlined, then the panic can be optimized out. Co-authored-by: Carl Lundin <[email protected]>
google · Nov 4, 2024 · c72fcbf · c72fcbf
1 parent 255babe
commit c72fcbf
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 0 deletions.
diff --git a/src/layout.rs b/src/layout.rs
@@ -451,6 +451,7 @@ impl DstLayout {
     /// rely on `validate_cast_and_convert_metadata` panicking in any particular
     /// condition, even if `debug_assertions` are enabled.
     #[allow(unused)]
+    #[inline(always)]
     pub(crate) const fn validate_cast_and_convert_metadata(
         &self,
         addr: usize,

diff --git a/src/pointer/ptr.rs b/src/pointer/ptr.rs
@@ -906,6 +906,7 @@ mod _casts {
         /// - If this is a prefix cast, `ptr` has the same address as `self`.
         /// - If this is a suffix cast, `remainder` has the same address as
         ///   `self`.
+        #[inline(always)]
         pub(crate) fn try_cast_into<U, R>(
             self,
             cast_type: CastType,