From 35f55764b81390a085fb90f624082c196fbd6229 Mon Sep 17 00:00:00 2001 From: mtklein Date: Wed, 8 Apr 2015 14:09:41 -0700 Subject: Revert of Rearrange SkRecord with small N in mind (patchset #8 id:120001 of https://codereview.chromium.org/1061783002/) Reason for revert: https://uberchromegw.corp.google.com/i/client.skia/builders/Test-Ubuntu-GCC-GCE-CPU-AVX2-x86-Debug/builds/149/steps/dm/logs/stdio Original issue's description: > Rearrange SkRecord with small N in mind > > This rearranges the record pointers and types so they can go in a single array, then preallocates some space for them and for the SkVarAlloc. > > picture_overhead_draw bench drops from ~1000ns to 500-600ns, with no effect on picture_overhead_nodraw. > > I don't see any significant effect on large picture recording times from our .skps. > > BUG=chromium:470553 > > Committed: https://skia.googlesource.com/skia/+/e2dd9408cd711777afaa9410427fb0d761ab004a TBR=reed@google.com,mtklein@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=chromium:470553 Review URL: https://codereview.chromium.org/1068383003 --- src/core/SkVarAlloc.h | 10 ---------- 1 file changed, 10 deletions(-) (limited to 'src/core/SkVarAlloc.h') diff --git a/src/core/SkVarAlloc.h b/src/core/SkVarAlloc.h index 8a55b36615..fb55192439 100644 --- a/src/core/SkVarAlloc.h +++ b/src/core/SkVarAlloc.h @@ -1,10 +1,3 @@ -/* - * Copyright 2015 Google Inc. - * - * Use of this source code is governed by a BSD-style license that can be - * found in the LICENSE file. - */ - #ifndef SkVarAlloc_DEFINED #define SkVarAlloc_DEFINED @@ -14,9 +7,6 @@ class SkVarAlloc : SkNoncopyable { public: // Smallest block we'll allocate is 2**N bytes. explicit SkVarAlloc(size_t minLgSize); - // Same as above, but first uses up to len bytes from storage. - SkVarAlloc(size_t minLgSize, char* storage, size_t len); - ~SkVarAlloc(); // Returns contiguous bytes aligned at least for pointers. You may pass SK_MALLOC_THROW, etc. -- cgit v1.2.3