diff options
author | Kostya Kortchinsky <kostyak@google.com> | 2021-04-14 21:09:20 -0700 |
---|---|---|
committer | Kostya Kortchinsky <kostyak@google.com> | 2021-04-15 13:26:23 -0700 |
commit | 3f97c66b00400f5e7f0cd260107f53c11b1cc33b (patch) | |
tree | 686dd2ec855ad35b670704c4d1a3a7b39c42277b /llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp | |
parent | 07edd789939238d309d2ab747fad79337d705909 (diff) | |
download | llvm-3f97c66b00400f5e7f0cd260107f53c11b1cc33b.zip llvm-3f97c66b00400f5e7f0cd260107f53c11b1cc33b.tar.gz llvm-3f97c66b00400f5e7f0cd260107f53c11b1cc33b.tar.bz2 |
[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
Diffstat (limited to 'llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp')
0 files changed, 0 insertions, 0 deletions