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

Issue 519: Added generic intrinsics abstraction layer for x86 and arm/neon.

Can't Edit
Can't Publish+Mail
Start Review
Created:
4 years, 1 month ago by tdaede
Modified:
3 years, 4 months ago
Reviewers:
derf
Visibility:
Public.

Patch Set 1 #

Total comments: 12

Patch Set 2 : Added generic intrinsics abstraction layer for x86 and arm/neon. #

Patch Set 3 : Added generic intrinsics abstraction layer for x86 and arm/neon. #

Unified diffs Side-by-side diffs Delta from patch set Stats Patch
M Makefile.am View 1 2 2 chunks +14 lines, -2 lines 0 comments Download
M configure.ac View 1 2 1 chunk +1 line, -1 line 0 comments Download
A src/simd/sad.c View 1 1 chunk +109 lines, -0 lines 0 comments Download
A src/simd/sad.h View 1 chunk +35 lines, -0 lines 0 comments Download
A src/simd/simd128.h View 1 2 1 chunk +34 lines, -0 lines 0 comments Download
A src/simd/simd64.h View 1 2 1 chunk +35 lines, -0 lines 0 comments Download
A src/simd/v128_intrinsics_arm.h View 1 2 1 chunk +397 lines, -0 lines 0 comments Download
A src/simd/v128_intrinsics_x86.h View 1 2 1 chunk +493 lines, -0 lines 0 comments Download
A src/simd/v64_intrinsics_arm.h View 1 2 1 chunk +461 lines, -0 lines 0 comments Download
A src/simd/v64_intrinsics_x86.h View 1 2 1 chunk +497 lines, -0 lines 0 comments Download
M unix/Makefile View 1 2 1 chunk +4 lines, -0 lines 0 comments Download

Messages

Total messages: 5
tdaede
4 years, 1 month ago #1
tdaede
Actually issue 422 but I am not allowed to add a patch to someone else's ...
4 years, 1 month ago #2
derf
r=me with nits fixed. http://review.xiph.org/519/diff/817/src/simd/simd128.h File src/simd/simd128.h (right): http://review.xiph.org/519/diff/817/src/simd/simd128.h#newcode28 src/simd/simd128.h:28: #ifdef __ARM_NEON__ I'll re-iterate that ...
4 years, 1 month ago #3
tdaede
3 years, 6 months ago #4
tdaede
3 years, 4 months ago #5

          
Sign in to reply to this message.

RSS Feeds Recent Issues | This issue
This is Rietveld