Page MenuHomePhabricator

scudo: Delete unused class TwoLevelByteMap. NFCI.
ClosedPublic

Authored by pcc on Feb 5 2020, 4:42 PM.

Details

Summary

The class is only used in SizeClassAllocator32 in 64-bit mode, but we don't
use that class in 64-bit mode.

Diff Detail

Event Timeline

pcc created this revision.Feb 5 2020, 4:42 PM
Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptFeb 5 2020, 4:42 PM
Herald added subscribers: Restricted Project, jfb. · View Herald Transcript
pcc retitled this revision from scudo: Delete unused class ScudoByteMap. NFCI. to scudo: Delete unused class TwoLevelByteMap. NFCI..Feb 5 2020, 4:43 PM

Unit tests: pass. 62519 tests passed, 0 failed and 844 were skipped.

clang-tidy: pass.

clang-format: fail. Please format your changes with clang-format by running git-clang-format HEAD^ or applying this patch.

Build artifacts: clang-tidy.txt, clang-format.patch, CMakeCache.txt, console-log.txt, test-results.xml

Pre-merge checks is in beta. Report issue. Please join beta or enable it for your project.

cryptoad accepted this revision.Feb 6 2020, 8:53 AM

I guess it can go! We'll bring it back if needed.

This revision is now accepted and ready to land.Feb 6 2020, 8:53 AM
This revision was automatically updated to reflect the committed changes.