Skip to content

Commit

Permalink
Merge pull request #383 from programmerjake/programmerjake-patch-1
Browse files Browse the repository at this point in the history
Fix load/store safety comments to require aligned `T`
  • Loading branch information
calebzulawski authored Dec 11, 2023
2 parents 80ff6a5 + c705775 commit 1273da6
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions crates/core_simd/src/vector.rs
Original file line number Diff line number Diff line change
Expand Up @@ -194,7 +194,7 @@ where
/// With padding, `read_unaligned` will read past the end of an array of N elements.
///
/// # Safety
/// Reading `ptr` must be safe, as if by `<*const [T; N]>::read_unaligned`.
/// Reading `ptr` must be safe, as if by `<*const [T; N]>::read`.
#[inline]
const unsafe fn load(ptr: *const [T; N]) -> Self {
// There are potentially simpler ways to write this function, but this should result in
Expand All @@ -215,7 +215,7 @@ where
/// See `load` as to why this function is necessary.
///
/// # Safety
/// Writing to `ptr` must be safe, as if by `<*mut [T; N]>::write_unaligned`.
/// Writing to `ptr` must be safe, as if by `<*mut [T; N]>::write`.
#[inline]
const unsafe fn store(self, ptr: *mut [T; N]) {
// There are potentially simpler ways to write this function, but this should result in
Expand Down

0 comments on commit 1273da6

Please sign in to comment.