This is an archive of the discontinued LLVM Phabricator instance.

[scudo] Simplify markFreeBlocks
ClosedPublic

Authored by Chia-hungDuan on Feb 3 2023, 4:02 PM.

Details

Summary

With memory group, we always mark the free blocks from the same region.
Therefore, we don't need to calculate the offset from base and determine
the region index. Also improve the way we deal with the last block in
the region so that the loop body is simpler.

Diff Detail

Event Timeline

Chia-hungDuan created this revision.Feb 3 2023, 4:02 PM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 3 2023, 4:02 PM
Chia-hungDuan requested review of this revision.Feb 3 2023, 4:02 PM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 3 2023, 4:02 PM
Herald added a subscriber: Restricted Project. · View Herald Transcript
cferris accepted this revision.Feb 16 2023, 5:39 PM

LGTM

This revision is now accepted and ready to land.Feb 16 2023, 5:39 PM

Rebase and fix one lint

This revision was landed with ongoing or failed builds.Mar 7 2023, 1:56 PM
This revision was automatically updated to reflect the committed changes.