From 4ca2947b0fe52708f2c057e32153685d0c0f2bda Mon Sep 17 00:00:00 2001 From: Paul Bone Date: Tue, 19 Mar 2024 01:42:41 +0000 Subject: [PATCH] Bug 1857841 - pt 2. Remove obsolete comment r=glandium Differential Revision: https://phabricator.services.mozilla.com/D200413 --- memory/build/mozjemalloc.cpp | 10 ---------- 1 file changed, 10 deletions(-) diff --git a/memory/build/mozjemalloc.cpp b/memory/build/mozjemalloc.cpp index 96872629d4d1..281cd788bc62 100644 --- a/memory/build/mozjemalloc.cpp +++ b/memory/build/mozjemalloc.cpp @@ -2694,16 +2694,6 @@ bool arena_t::SplitRun(arena_run_t* aRun, size_t aSize, bool aLarge, void arena_t::InitChunk(arena_chunk_t* aChunk) { size_t i; - // WARNING: The following relies on !aZeroed meaning "used to be an arena - // chunk". - // When the chunk we're initializating as an arena chunk is zeroed, we - // mark all runs are decommitted and zeroed. - // When it is not, which we can assume means it's a recycled arena chunk, - // all it can contain is an arena chunk header (which we're overwriting), - // and zeroed or poisoned memory (because a recycled arena chunk will - // have been emptied before being recycled). In that case, we can get - // away with reusing the chunk as-is, marking all runs as madvised. - size_t flags = CHUNK_MAP_DECOMMITTED | CHUNK_MAP_ZEROED; mStats.mapped += kChunkSize;