Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(50)

Issue 4365051: Remove Redundant functions for arm memset optimizations (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
13 years, 8 months ago by Shankar
Modified:
13 years, 8 months ago
Base URL:
http://skia.googlecode.com/svn/trunk/
Visibility:
Public.

Description

Unnecessary code duplication for returning arm/neon version of memset . Removed the redundant file and merged the functionality into opts_check_arm_neon.cpp which will return arm or neon version of memset based on build flag

Patch Set 1 #

Patch Set 2 : Dropped opts_check_arm_neon.cpp and merged functionality to opts_check_arm.cpp as per Steve advice #

Unified diffs Side-by-side diffs Delta from patch set Stats (+52 lines, -65 lines) Patch
D src/opts/opts_check_arm.cpp View 1 1 chunk +52 lines, -20 lines 0 comments Download
M src/opts/opts_check_arm_neon.cpp View 1 1 chunk +0 lines, -45 lines 0 comments Download

Messages

Total messages: 11
Shankar
This patch will remove redundant functions for ARM memset routines in skia library.
13 years, 8 months ago (2011-04-12 08:28:25 UTC) #1
agl
13 years, 8 months ago (2011-04-12 14:27:46 UTC) #2
piman
13 years, 8 months ago (2011-04-13 06:56:39 UTC) #3
reed1
LGTM
13 years, 8 months ago (2011-04-13 11:39:16 UTC) #4
steve.mcintyre
On 2011/04/12 08:28:25, Shankar wrote: > This patch will remove redundant functions for ARM memset ...
13 years, 8 months ago (2011-04-13 11:52:05 UTC) #5
Shankar
> > Might sound like nit-picking, but I'd change very slightly here - move the ...
13 years, 8 months ago (2011-04-14 10:23:38 UTC) #6
Shankar
New Patchset uploaded, incorporating Steve's comments
13 years, 8 months ago (2011-04-18 11:41:55 UTC) #7
Shankar
Hi Reviewers, Could you please commit my changes if they LG ...
13 years, 8 months ago (2011-04-27 08:34:54 UTC) #8
reed1
LGTM
13 years, 8 months ago (2011-04-27 11:42:40 UTC) #9
reed1
Committed revision 1197.
13 years, 8 months ago (2011-04-27 11:49:09 UTC) #10
Shankar
13 years, 8 months ago (2011-04-27 12:32:27 UTC) #11
Closing the issue
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b