Bug 100737 - Optimize vclip for NEON in VectorMath
Summary: Optimize vclip for NEON in VectorMath
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Audio (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-10-30 02:00 PDT by Gabor Rapcsanyi
Modified: 2013-02-26 21:43 PST (History)
6 users (show)

See Also:

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

Note You need to log in before you can comment on or make changes to this bug.
Description Gabor Rapcsanyi 2012-10-30 02:00:00 PDT 
Optimize vclip for NEON in VectorMath.
Comment 1 Gabor Rapcsanyi 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
Comment 2 WebKit Review Bot 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>
Comment 3 WebKit Review Bot 2012-10-30 03:51:05 PDT 
All reviewed patches have been landed.  Closing bug.
Comment 4 kdj 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
Comment 5 Gabor Rapcsanyi 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 :)