commit 1da26c29a29be5b15b859dcbad1c611f0022f1c6
author Kostya Kortchinsky <kostyak@google.com> Wed Apr 14 21:09:20 2021 -0700
committer Copybara-Service <copybara-worker@google.com> Fri Apr 16 13:21:55 2021 -0700
tree 40344dcabc79af0500e594b5b44527d7f9a996a6
parent d3845f12974dac72aa1da66950c46bdc987f459d

[scudo][standalone] Fuchsia related fixes

While attempting to roll the latest Scudo in Fuchsia, some issues
arose. While trying to debug them, it appeared that `DCHECK`s were
also never exercised in Fuchsia. This CL fixes the following
problems:
- the size of a block in the TransferBatch class must be a multiple
  of the compact pointer scale. In some cases, it wasn't true, which
  lead to obscure crashes. Now, we round up `sizeof(TransferBatch)`.
  This only materialized in Fuchsia due to the specific parameters
  of the `DefaultConfig`;
- 2 `DCHECK` statements in Fuchsia were incorrect;
- `map()` & co. require a size multiple of a page (as enforced in
  Fuchsia `DCHECK`s), which wasn't the case for `PackedCounters`.
- In the Secondary, a parameter was marked as `UNUSED` while it is
  actually used.

Differential Revision: https://reviews.llvm.org/D100524

GitOrigin-RevId: 3f97c66b00400f5e7f0cd260107f53c11b1cc33b