This is an archive of the discontinued LLVM Phabricator instance.

[scudo][standalone] Change default Android config
ClosedPublic

Authored by cryptoad on Jan 30 2020, 3:27 PM.

Details

Summary

This changessome of the parameters related to the default Android config
to address some performance and memory footprint issues (to be closer
to the default Bionic allocator numbers).

Diff Detail

Event Timeline

cryptoad created this revision.Jan 30 2020, 3:27 PM
Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptJan 30 2020, 3:27 PM
Herald added subscribers: llvm-commits, Restricted Project. · View Herald Transcript

Unit tests: fail. 62354 tests passed, 1 failed and 839 were skipped.

failed: libc++.std/containers/sequences/array/array_creation/to_array.fail.cpp

clang-tidy: pass.

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 updated this revision to Diff 241772.Jan 31 2020, 10:50 AM

Updated some parameters.

Unit tests: fail. 62354 tests passed, 1 failed and 839 were skipped.

failed: libc++.std/containers/sequences/array/array_creation/to_array.fail.cpp

clang-tidy: pass.

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 edited the summary of this revision. (Show Details)Jan 31 2020, 2:04 PM
hctim accepted this revision.Jan 31 2020, 2:13 PM
This revision is now accepted and ready to land.Jan 31 2020, 2:13 PM
pcc accepted this revision.Jan 31 2020, 2:44 PM

LGTM

This revision was automatically updated to reflect the committed changes.