This is an archive of the discontinued LLVM Phabricator instance.

scudo: Use more size classes in the malloc_free_loop benchmarks.
ClosedPublic

Authored by pcc on Feb 3 2020, 12:43 PM.

Details

Summary

As a result of recent changes to the Android size classes, the malloc_free_loop
benchmark started exhausing the 8192 size class at 32768 iterations. To avoid
this problem (and to make the test more realistic), change the benchmark to
use a variety of size classes.

Diff Detail

Event Timeline

pcc created this revision.Feb 3 2020, 12:43 PM
Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptFeb 3 2020, 12:43 PM
Herald added a subscriber: Restricted Project. · View Herald Transcript

Unit tests: pass. 62422 tests passed, 0 failed and 845 were skipped.

clang-tidy: fail. clang-tidy found 1 errors and 0 warnings. 0 of them are added as review comments below (why?).

clang-format: pass.

Build artifacts: diff.json, 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 3 2020, 1:09 PM
This revision is now accepted and ready to land.Feb 3 2020, 1:09 PM
This revision was automatically updated to reflect the committed changes.