diff options
| author |  mtklein <mtklein@chromium.org> | 2016-01-05 05:06:26 -0800 |
|---|---|---|
| committer |  Commit bot <commit-bot@chromium.org> | 2016-01-05 05:06:26 -0800 |
| commit | 2c2d8ab08c39fad2706895e69dcc9ed1aeb9bb2a (patch) | |
| tree | 918145ceafba91256e6c426b504b3ee8c84e30fe /src/core/SkBitmapProcState.h | |
| parent | 68bb45ea83c929870c9fc33c8566ae298113ed83 (diff) | |
df generation: single allocation with calloc
The dfStorage DFData allocation can never fit in its stack space: 5px padding on each side always implies at least a 10x10 DFData allocation, but the stack space only fits 64 DFData.
So we've always been spilling to the heap.
If we're going to spill to the heap, we might as well allocate/free all our temporary memory in one block, and since we want it zeroed, might as well calloc.
So in practice this replaces 1-2 malloc, 1-2 free, and 2 bzeros with 1 calloc and 1 free.
BUG=skia:4729
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1544983004
Committed: https://skia.googlesource.com/skia/+/12204d90337656542a42fa0fcccb7bec13af0cce
CQ_EXTRA_TRYBOTS=client.skia.android:Test-Android-GCC-Nexus6-GPU-Adreno420-Arm7-Release-Trybot
Review URL: https://codereview.chromium.org/1544983004
Diffstat (limited to 'src/core/SkBitmapProcState.h')
0 files changed, 0 insertions, 0 deletions