This is an archive of the discontinued LLVM Phabricator instance.

[scudo] Fix RingBuffer initialization.
ClosedPublic

Authored by supersymetrie on Jan 12 2023, 12:58 PM.

Details

Summary

Several map implementations require that map calls provide a page
aligned size.

Diff Detail

Event Timeline

supersymetrie created this revision.Jan 12 2023, 12:58 PM
Herald added a project: Restricted Project. · View Herald TranscriptJan 12 2023, 12:58 PM
Herald added a subscriber: Enna1. · View Herald Transcript
supersymetrie requested review of this revision.Jan 12 2023, 12:58 PM
Herald added a project: Restricted Project. · View Herald TranscriptJan 12 2023, 12:58 PM
Herald added a subscriber: Restricted Project. · View Herald Transcript
fmayer accepted this revision.Jan 12 2023, 1:00 PM
fmayer added a subscriber: fmayer.

LGTM, thanks.

Not in this CL: If we want to keep map requiring aligned sizes, I suggest we put a DCHECK into the linux one as well.

This revision is now accepted and ready to land.Jan 12 2023, 1:00 PM
This revision was landed with ongoing or failed builds.Jan 12 2023, 1:10 PM
This revision was automatically updated to reflect the committed changes.