Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1921)

Issue 16004011: Added megalooper GM (Closed)

Created:
7 years, 6 months ago by robertphillips
Modified:
7 years, 6 months ago
Reviewers:
bsalomon, reed1
CC:
skia-review_googlegroups.com, jbroman1
Visibility:
Public.

Description

Here are the sizes (of the skps on disk): megalooper_0x0: 17KB megalooper_1x5: 10KB megalooper_4x2: 38KB Here are the GPU times (with 256x256 tiling w/ bbh grid): megalooper_0x0: 300.68 megalooper_1x5: 348.67 megalooper_4x2: 330.23 Here are the raster times (same setup): megalooper_0x0: 91.65 megalooper_1x5: 125.75 megalooper_4x2: 94.11 Here are the times w/o the trailing "original" rect: gpu: megalooper_0x0: 300.85 megalooper_1x4: 344.24 megalooper_4x1: 314.13 raster: megalooper_0x0: 92.72 megalooper_1x4: 125.89 megalooper_4x1: 92.44

Patch Set 1 #

Patch Set 2 : Actually added new GM file #

Patch Set 3 : Added comments #

Patch Set 4 : Cleaned up & converted to 1x4 & 4x1 (instead of 1x4 and 4x2) #

Unified diffs Side-by-side diffs Delta from patch set Stats (+251 lines, -0 lines) Patch
A gm/megalooper.cpp View 1 2 3 1 chunk +250 lines, -0 lines 0 comments Download
M gyp/gmslides.gypi View 1 2 3 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
robertphillips
7 years, 6 months ago (2013-05-31 17:44:39 UTC) #1
robertphillips
I actually would like to add this as a GM to capture mega-looper regressions as ...
7 years, 6 months ago (2013-06-03 16:24:58 UTC) #2
bsalomon
On 2013/06/03 16:24:58, robertphillips wrote: > I actually would like to add this as a ...
7 years, 6 months ago (2013-06-03 16:42:01 UTC) #3
robertphillips
7 years, 6 months ago (2013-06-03 17:13:39 UTC) #4
Message was sent while issue was closed.
committed as r9409

Powered by Google App Engine
This is Rietveld 408576698