Skip to content

Commit 238f563

Browse files
ckennellycopybara-github
authored andcommitted
Remove TODO.
While it avoids some chases, it makes the bookkeeping more complex and subsequent lookups are in L1. PiperOrigin-RevId: 955397117
1 parent b343eb8 commit 238f563

1 file changed

Lines changed: 0 additions & 3 deletions

File tree

tcmalloc/central_freelist.h

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -628,9 +628,6 @@ inline void CentralFreeList<Forwarder>::InsertRange(absl::Span<void*> batch) {
628628
//
629629
// We only look for consecutive runs of the same span to avoid the cost of
630630
// sorting, solely for telemetry purposes.
631-
//
632-
// TODO(b/175159154): Consider doing this before MapObjectsToSpans based on a
633-
// ~(kPageSize - 1u) pointer mask heuristic.
634631
const int same_span =
635632
central_freelist_internal::RecordSameSpanRuns<kMaxObjectsToMove>(
636633
absl::MakeConstSpan(spans, batch.size()),

0 commit comments

Comments
 (0)