refactor(allocator): inline bumpalo into oxc_allocator

Cargo.toml

@@ -170,11 +170,6 @@ allocator-api2 = "=0.2.21" # Allocator trait
 base64 = "0.22.1" # Base64 encoding
 bitflags = "2.10.0" # Bitflag types
 bpaf = "0.9.20" # CLI parser
-# `bumpalo` must be pinned to exactly version 3.19.0.
-# `Allocator::from_raw_parts` (used in raw transfer) depends on internal implementation details
-# of `bumpalo` which may change in a future version.
-# This is a temporary situation - we'll replace `bumpalo` with our own allocator.
-bumpalo = "=3.19.1" # Bump allocator
 compact_str = "0.9.0" # Compact string type
 console = "0.16.2" # Terminal utilities
 constcat = "0.6.1" # Const string concatenation

crates/oxc_allocator/Cargo.toml

@@ -25,7 +25,6 @@ oxc_data_structures = { workspace = true, features = ["assert_unchecked", "stack
 oxc_estree = { workspace = true, optional = true }
 
 allocator-api2 = { workspace = true }
-bumpalo = { workspace = true, features = ["allocator-api2", "collections"] }
 hashbrown = { workspace = true, default-features = false, features = [
   "inline-more",
   "allocator-api2",

crates/oxc_allocator/src/alloc.rs

@@ -7,7 +7,8 @@ use std::{
 };
 
 use allocator_api2::alloc::Allocator;
-use bumpalo::Bump;
+
+use crate::bump::Bump;
 
 /// Trait describing an allocator.
 ///
@@ -81,7 +82,7 @@ pub trait Alloc {
     ) -> NonNull<u8>;
 }
 
-/// Implement [`Alloc`] for [`bumpalo::Bump`].
+/// Implement [`Alloc`] for [`Bump`].
 ///
 /// All methods except `alloc` delegate to [`Bump`]'s impl of `allocator_api2`'s [`Allocator`] trait.
 impl Alloc for Bump {
@@ -114,7 +115,7 @@ impl Alloc for Bump {
     #[inline(always)]
     unsafe fn dealloc(&self, ptr: NonNull<u8>, layout: Layout) {
         // SAFETY: Safety requirements of `Allocator::deallocate` are the same as for this method
-        unsafe { self.deallocate(ptr, layout) }
+        unsafe { Allocator::deallocate(&self, ptr, layout) }
     }
 
     /// Grow an existing allocation to new [`Layout`].

crates/oxc_allocator/src/allocator.rs

@@ -7,7 +7,7 @@ use std::{
 #[cfg(all(feature = "track_allocations", not(feature = "disable_track_allocations")))]
 use std::mem::offset_of;
 
-use bumpalo::Bump;
+use crate::bump::Bump;
 
 use oxc_data_structures::assert_unchecked;
 
@@ -613,7 +613,7 @@ impl Allocator {
         bytes
     }
 
-    /// Get inner [`bumpalo::Bump`].
+    /// Get inner [`Bump`].
     ///
     /// This method is not public. We don't want to expose `Bump` to user.
     /// The fact that we're using `bumpalo` is an internal implementation detail.
@@ -625,7 +625,7 @@ impl Allocator {
         &self.bump
     }
 
-    /// Create [`Allocator`] from a [`bumpalo::Bump`].
+    /// Create [`Allocator`] from a [`Bump`].
     ///
     /// This method is not public. Only used by [`Allocator::from_raw_parts`].
     //

crates/oxc_allocator/src/allocator_api2.rs

@@ -1,23 +1,22 @@
-// All methods just delegate to `bumpalo`, so all marked `#[inline(always)]`.
-// All have same safety preconditions of `bumpalo` methods of the same name.
+// All methods just delegate to `Bump`, so all marked `#[inline(always)]`.
+// All have same safety preconditions of `Bump` methods of the same name.
 #![expect(clippy::inline_always, clippy::undocumented_unsafe_blocks)]
 
 use std::{alloc::Layout, ptr::NonNull};
 
 use allocator_api2::alloc::{AllocError, Allocator};
 
-/// SAFETY:
-/// <https://github.com/fitzgen/bumpalo/blob/4eeab8847c85d5cde135ca21ae14a54e56b05224/src/lib.rs#L1938>
+/// SAFETY: See `bump.rs` for the implementation of `Allocator` for `&Bump`.
 unsafe impl Allocator for &crate::Allocator {
     #[inline(always)]
     fn allocate(&self, layout: Layout) -> Result<NonNull<[u8]>, AllocError> {
-        self.bump().allocate(layout)
+        Allocator::allocate(&self.bump(), layout)
     }
 
     #[inline(always)]
     unsafe fn deallocate(&self, ptr: NonNull<u8>, layout: Layout) {
         unsafe {
-            self.bump().deallocate(ptr, layout);
+            Allocator::deallocate(&self.bump(), ptr, layout);
         }
     }
 
@@ -28,7 +27,7 @@ unsafe impl Allocator for &crate::Allocator {
         old_layout: Layout,
         new_layout: Layout,
     ) -> Result<NonNull<[u8]>, AllocError> {
-        unsafe { self.bump().shrink(ptr, old_layout, new_layout) }
+        unsafe { Allocator::shrink(&self.bump(), ptr, old_layout, new_layout) }
     }
 
     #[inline(always)]
@@ -38,7 +37,7 @@ unsafe impl Allocator for &crate::Allocator {
         old_layout: Layout,
         new_layout: Layout,
     ) -> Result<NonNull<[u8]>, AllocError> {
-        unsafe { self.bump().grow(ptr, old_layout, new_layout) }
+        unsafe { Allocator::grow(&self.bump(), ptr, old_layout, new_layout) }
     }
 
     #[inline(always)]
@@ -48,6 +47,6 @@ unsafe impl Allocator for &crate::Allocator {
         old_layout: Layout,
         new_layout: Layout,
     ) -> Result<NonNull<[u8]>, AllocError> {
-        unsafe { self.bump().grow_zeroed(ptr, old_layout, new_layout) }
+        unsafe { Allocator::grow_zeroed(&self.bump(), ptr, old_layout, new_layout) }
     }
 }