Bug 100737

Summary:

Optimize vclip for NEON in VectorMath

Product:

WebKit

Reporter:

Gabor Rapcsanyi <rgabor>

Component:

Web Audio

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

crogers, eric.carlson, feature-media-reviews, kdj.tikka, s.choi, webkit.review.bot

Priority:

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
proposed patch	none

Gabor Rapcsanyi

Reported 2012-10-30 02:00:00 PDT

Optimize vclip for NEON in VectorMath.

Attachments
proposed patch (1.75 KB, patch) 2012-10-30 02:40 PDT, Gabor Rapcsanyi	no flags	Details Formatted Diff Diff
View All Add attachment proposed patch, testcase, etc.

Gabor Rapcsanyi

Comment 1 2012-10-30 02:40:59 PDT

Created attachment 171404 [details] proposed patch I have made independent tests on this function. The vector size was 1000 with float numbers and I ran it 100 000 times. The average results: REF elapsed time is 1219 ms NEON elapsed time is 427 ms

WebKit Review Bot

Comment 2 2012-10-30 03:51:01 PDT

Comment on attachment 171404 [details] proposed patch Clearing flags on attachment: 171404 Committed r132890: <http://trac.webkit.org/changeset/132890>

WebKit Review Bot

Comment 3 2012-10-30 03:51:05 PDT

All reviewed patches have been landed. Closing bug.

kdj

Comment 4 2013-02-24 23:18:45 PST

Hi all, Here on top of this patch i would like to submit more optimized ARM-NEON solution. Please let me know, is it OK to submit here ? or should i create new bug. Regards, kdj

Gabor Rapcsanyi

Comment 5 2013-02-24 23:27:37 PST

(In reply to comment #4) > Hi all, > > Here on top of this patch i would like to submit more optimized ARM-NEON solution. > > Please let me know, is it OK to submit here ? > or should i create new bug. > > Regards, > kdj Hello! That's good but this patch landed so you should make another bug for yours. Please CC me as well :)

Note You need to log in before you can comment on or make changes to this bug.