align_offset, align_to: no longer allow implementations to spuriously fail to align

This commit is contained in:
Ralf Jung 2024-02-16 21:22:47 +01:00
parent d2e8ecd8bd
commit 507583a40c
3 changed files with 24 additions and 16 deletions

View File

@ -1322,9 +1322,7 @@ impl<T: ?Sized> *const T {
/// `align`. /// `align`.
/// ///
/// If it is not possible to align the pointer, the implementation returns /// If it is not possible to align the pointer, the implementation returns
/// `usize::MAX`. It is permissible for the implementation to *always* /// `usize::MAX`.
/// return `usize::MAX`. Only your algorithm's performance can depend
/// on getting a usable offset here, not its correctness.
/// ///
/// The offset is expressed in number of `T` elements, and not bytes. The value returned can be /// The offset is expressed in number of `T` elements, and not bytes. The value returned can be
/// used with the `wrapping_add` method. /// used with the `wrapping_add` method.
@ -1333,6 +1331,15 @@ impl<T: ?Sized> *const T {
/// beyond the allocation that the pointer points into. It is up to the caller to ensure that /// beyond the allocation that the pointer points into. It is up to the caller to ensure that
/// the returned offset is correct in all terms other than alignment. /// the returned offset is correct in all terms other than alignment.
/// ///
/// When this is called during compile-time evaluation (which is unstable), the implementation
/// may return `usize::MAX` in cases where that can never happen at runtime. This is because the
/// actual alignment of pointers is not known yet during compile-time, so an offset with
/// guaranteed alignment can sometimes not be computed. For example, a buffer declared as `[u8;
/// N]` might be allocated at an odd or an even address, but at compile-time this is not yet
/// known, so the execution has to be correct for either choice. It is therefore impossible to
/// find an offset that is guaranteed to be 2-aligned. (This behavior is subject to change, as usual
/// for unstable APIs.)
///
/// # Panics /// # Panics
/// ///
/// The function panics if `align` is not a power-of-two. /// The function panics if `align` is not a power-of-two.

View File

@ -1585,9 +1585,7 @@ impl<T: ?Sized> *mut T {
/// `align`. /// `align`.
/// ///
/// If it is not possible to align the pointer, the implementation returns /// If it is not possible to align the pointer, the implementation returns
/// `usize::MAX`. It is permissible for the implementation to *always* /// `usize::MAX`.
/// return `usize::MAX`. Only your algorithm's performance can depend
/// on getting a usable offset here, not its correctness.
/// ///
/// The offset is expressed in number of `T` elements, and not bytes. The value returned can be /// The offset is expressed in number of `T` elements, and not bytes. The value returned can be
/// used with the `wrapping_add` method. /// used with the `wrapping_add` method.
@ -1596,6 +1594,15 @@ impl<T: ?Sized> *mut T {
/// beyond the allocation that the pointer points into. It is up to the caller to ensure that /// beyond the allocation that the pointer points into. It is up to the caller to ensure that
/// the returned offset is correct in all terms other than alignment. /// the returned offset is correct in all terms other than alignment.
/// ///
/// When this is called during compile-time evaluation (which is unstable), the implementation
/// may return `usize::MAX` in cases where that can never happen at runtime. This is because the
/// actual alignment of pointers is not known yet during compile-time, so an offset with
/// guaranteed alignment can sometimes not be computed. For example, a buffer declared as `[u8;
/// N]` might be allocated at an odd or an even address, but at compile-time this is not yet
/// known, so the execution has to be correct for either choice. It is therefore impossible to
/// find an offset that is guaranteed to be 2-aligned. (This behavior is subject to change, as usual
/// for unstable APIs.)
///
/// # Panics /// # Panics
/// ///
/// The function panics if `align` is not a power-of-two. /// The function panics if `align` is not a power-of-two.

View File

@ -3756,11 +3756,8 @@ impl<T> [T] {
/// maintained. /// maintained.
/// ///
/// This method splits the slice into three distinct slices: prefix, correctly aligned middle /// This method splits the slice into three distinct slices: prefix, correctly aligned middle
/// slice of a new type, and the suffix slice. How exactly the slice is split up is not /// slice of a new type, and the suffix slice. The middle part will be as big as possible under
/// specified; the middle part may be smaller than necessary. However, if this fails to return a /// the given alignment constraint and element size.
/// maximal middle part, that is because code is running in a context where performance does not
/// matter, such as a sanitizer attempting to find alignment bugs. Regular code running
/// in a default (debug or release) execution *will* return a maximal middle part.
/// ///
/// This method has no purpose when either input element `T` or output element `U` are /// This method has no purpose when either input element `T` or output element `U` are
/// zero-sized and will return the original slice without splitting anything. /// zero-sized and will return the original slice without splitting anything.
@ -3824,11 +3821,8 @@ impl<T> [T] {
/// types is maintained. /// types is maintained.
/// ///
/// This method splits the slice into three distinct slices: prefix, correctly aligned middle /// This method splits the slice into three distinct slices: prefix, correctly aligned middle
/// slice of a new type, and the suffix slice. How exactly the slice is split up is not /// slice of a new type, and the suffix slice. The middle part will be as big as possible under
/// specified; the middle part may be smaller than necessary. However, if this fails to return a /// the given alignment constraint and element size.
/// maximal middle part, that is because code is running in a context where performance does not
/// matter, such as a sanitizer attempting to find alignment bugs. Regular code running
/// in a default (debug or release) execution *will* return a maximal middle part.
/// ///
/// This method has no purpose when either input element `T` or output element `U` are /// This method has no purpose when either input element `T` or output element `U` are
/// zero-sized and will return the original slice without splitting anything. /// zero-sized and will return the original slice without splitting anything.