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

Issue 21931002: ARM Skia NEON patches - 18 - Preparation work for BitmapProcState (Closed)

Created:
7 years, 4 months ago by kevin.petit.not.used.account
Modified:
7 years, 3 months ago
Reviewers:
djsollen
CC:
skia-review_googlegroups.com
Visibility:
Public.

Description

ARM Skia NEON patches - 18 - Preparation work for BitmapProcState BitmapProcState: clean a little and get rid of some asm replacing the apparently stupid dx+dx+dx leads to more instructions being generated. Signed-off-by: Kévin PETIT <kevin.petit@arm.com>; BitmapProcState: move code common to C and NEON to a separate header Signed-off-by: Kévin PETIT <kevin.petit@arm.com>; BUG= Committed: http://code.google.com/p/skia/source/detail?r=11109

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+74 lines, -93 lines) Patch
M src/core/SkBitmapProcState_matrixProcs.cpp View 2 chunks +1 line, -36 lines 0 comments Download
A src/core/SkBitmapProcState_utils.h View 1 chunk +40 lines, -0 lines 0 comments Download
M src/opts/SkBitmapProcState_matrixProcs_neon.cpp View 3 chunks +33 lines, -57 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
kevin.petit.not.used.account
7 years, 3 months ago (2013-08-28 16:58:22 UTC) #1
kevin.petit.not.used.account
ping. This is currently preventing me from uploading a big refactoring of the BitmapProcState NEON ...
7 years, 3 months ago (2013-09-04 17:05:09 UTC) #2
djsollen
lgtm
7 years, 3 months ago (2013-09-04 19:29:23 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://skia-tree-status.appspot.com/cq/kevin.petit.arm@gmail.com/21931002/1
7 years, 3 months ago (2013-09-05 10:07:02 UTC) #4
commit-bot: I haz the power
7 years, 3 months ago (2013-09-05 18:28:03 UTC) #5
Message was sent while issue was closed.
Change committed as 11109

Powered by Google App Engine
This is Rietveld 408576698