From 4d0b39fb54014fb49aa0008d1fa99b94ae6f9dc1 Mon Sep 17 00:00:00 2001
From: overlookmotel <557937+overlookmotel@users.noreply.github.com>
Date: Thu, 9 Apr 2026 08:45:25 +0000
Subject: [PATCH] refactor(allocator): fix `ref_as_ptr` clippy warnings
 (#21220)

Continuation of #21210.

Replace `x as *const T` /  `x as *mut T` with `ptr::from_ref(x)` / `ptr::from_mut(x)`. Note that this change does not materialize any references which did not already exist in original code.
---
 crates/oxc_allocator/src/bump.rs | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/crates/oxc_allocator/src/bump.rs b/crates/oxc_allocator/src/bump.rs
index 8831d24b41e6b..9a8b23ae2db7b 100644
--- a/crates/oxc_allocator/src/bump.rs
+++ b/crates/oxc_allocator/src/bump.rs
@@ -12,7 +12,6 @@
     clippy::mut_from_ref,
     clippy::ptr_as_ptr,
     clippy::ptr_cast_constness,
-    clippy::ref_as_ptr,
     clippy::undocumented_unsafe_blocks,
     clippy::unnecessary_safety_comment,
     unsafe_op_in_unsafe_fn
@@ -374,8 +373,9 @@ impl ChunkFooter {
         let data = self.data.as_ptr() as *const u8;
         let ptr = self.ptr.get().as_ptr() as *const u8;
         debug_assert!(data <= ptr);
-        debug_assert!(ptr <= self as *const ChunkFooter as *const u8);
-        let len = unsafe { (self as *const ChunkFooter as *const u8).offset_from(ptr) as usize };
+        debug_assert!(ptr <= ptr::from_ref::<ChunkFooter>(self) as *const u8);
+        let len =
+            unsafe { (ptr::from_ref::<ChunkFooter>(self) as *const u8).offset_from(ptr) as usize };
         (ptr, len)
     }
 
@@ -1219,7 +1219,7 @@ impl<const MIN_ALIGN: usize> Bump<MIN_ALIGN> {
                 // since it grows backwards, it seems unlikely that we'd get
                 // any more than the `Result`'s discriminant this way, if
                 // anything at all.
-                &mut *(t as *mut _)
+                &mut *ptr::from_mut(t)
             }),
             Err(e) => unsafe {
                 // If this result was the last allocation in this arena, we can
@@ -1264,7 +1264,7 @@ impl<const MIN_ALIGN: usize> Bump<MIN_ALIGN> {
                 //
                 // The order between this and the deallocation doesn't matter
                 // because `Self: !Sync`.
-                Err(ptr::read(e as *const _))
+                Err(ptr::read(ptr::from_ref(e)))
             },
         }
     }
@@ -1329,7 +1329,7 @@ impl<const MIN_ALIGN: usize> Bump<MIN_ALIGN> {
                 // since it grows backwards, it seems unlikely that we'd get
                 // any more than the `Result`'s discriminant this way, if
                 // anything at all.
-                &mut *(t as *mut _)
+                &mut *ptr::from_mut(t)
             }),
             Err(e) => unsafe {
                 // If this result was the last allocation in this arena, we can
@@ -1374,7 +1374,7 @@ impl<const MIN_ALIGN: usize> Bump<MIN_ALIGN> {
                 //
                 // The order between this and the deallocation doesn't matter
                 // because `Self: !Sync`.
-                Err(AllocOrInitError::Init(ptr::read(e as *const _)))
+                Err(AllocOrInitError::Init(ptr::read(ptr::from_ref(e))))
             },
         }
     }