[PATCH] D101105: scudo: Optimize getSizeLSBByClassId() by compressing the table into an integer if possible. NFCI.

Peter Collingbourne via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 22 15:35:31 PDT 2021


pcc added a comment.

In D101105#2710053 <https://reviews.llvm.org/D101105#2710053>, @eugenis wrote:

> Nice.
> If this is a significant optimization, we might want to request it through the allocator config and fail if it is impossible - in case we change the size class map in the future and break this by accident.

Sure, it makes sense to have a check for the Android size class map. It seems like the most straightforward way to do it would be to expose the bool and check it via a static_assert, which I've done.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D101105/new/

https://reviews.llvm.org/D101105



More information about the llvm-commits mailing list