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

Side by Side Diff: src/opts/SkBlitRow_opts_SSE4.cpp

Issue 874033004: Revert of SSE4 opaque blend using intrinsics instead of assembly. (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Created 5 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « src/opts/SkBlitRow_opts_SSE4.h ('k') | src/opts/SkBlitRow_opts_SSE4_asm.S » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
(Empty)
1 #include "SkBlitRow_opts_SSE4.h"
2
3 // Some compilers can't compile SSSE3 or SSE4 intrinsics. We give them stub met hods.
4 // The stubs should never be called, so we make them crash just to confirm that.
5 #if SK_CPU_SSE_LEVEL < SK_CPU_SSE_LEVEL_SSE41
6 void S32A_Opaque_BlitRow32_SSE4(SkPMColor* SK_RESTRICT, const SkPMColor* SK_REST RICT, int, U8CPU) {
7 sk_throw();
8 }
9
10 #else
11
12 #ifdef _MSC_VER
13 #include <intrin.h>
14 #else
15 #include <x86intrin.h>
16 #endif
17
18 #include "SkColorPriv.h"
19 #include "SkColor_opts_SSE2.h"
20
21 void S32A_Opaque_BlitRow32_SSE4(SkPMColor* SK_RESTRICT dst,
22 const SkPMColor* SK_RESTRICT src,
23 int count,
24 U8CPU alpha) {
25 SkASSERT(alpha == 255);
26 // As long as we can, we'll work on 16 pixel pairs at once.
27 int count16 = count / 16;
28 __m128i* dst4 = (__m128i*)dst;
29 const __m128i* src4 = (const __m128i*)src;
30
31 for (int i = 0; i < count16 * 4; i += 4) {
32 // Load 16 source pixels.
33 __m128i s0 = _mm_loadu_si128(src4+i+0),
34 s1 = _mm_loadu_si128(src4+i+1),
35 s2 = _mm_loadu_si128(src4+i+2),
36 s3 = _mm_loadu_si128(src4+i+3);
37
38 const __m128i alphaMask = _mm_set1_epi32(0xFF << SK_A32_SHIFT);
39 const __m128i ORed = _mm_or_si128(s3, _mm_or_si128(s2, _mm_or_si128(s1, s0)));
40 if (_mm_testz_si128(ORed, alphaMask)) {
41 // All 16 source pixels are fully transparent. There's nothing to d o!
42 continue;
43 }
44 const __m128i ANDed = _mm_and_si128(s3, _mm_and_si128(s2, _mm_and_si128( s1, s0)));
45 if (_mm_testc_si128(ANDed, alphaMask)) {
46 // All 16 source pixels are fully opaque. There's no need to read d st or blend it.
47 _mm_storeu_si128(dst4+i+0, s0);
48 _mm_storeu_si128(dst4+i+1, s1);
49 _mm_storeu_si128(dst4+i+2, s2);
50 _mm_storeu_si128(dst4+i+3, s3);
51 continue;
52 }
53 // The general slow case: do the blend for all 16 pixels.
54 _mm_storeu_si128(dst4+i+0, SkPMSrcOver_SSE2(s0, _mm_loadu_si128(dst4+i+0 )));
55 _mm_storeu_si128(dst4+i+1, SkPMSrcOver_SSE2(s1, _mm_loadu_si128(dst4+i+1 )));
56 _mm_storeu_si128(dst4+i+2, SkPMSrcOver_SSE2(s2, _mm_loadu_si128(dst4+i+2 )));
57 _mm_storeu_si128(dst4+i+3, SkPMSrcOver_SSE2(s3, _mm_loadu_si128(dst4+i+3 )));
58 }
59
60 // Wrap up the last <= 15 pixels.
61 for (int i = count16*16; i < count; i++) {
62 // This check is not really necessarily, but it prevents pointless autov ectorization.
63 if (src[i] & 0xFF000000) {
64 dst[i] = SkPMSrcOver(src[i], dst[i]);
65 }
66 }
67 }
68
69 #endif
OLDNEW
« no previous file with comments | « src/opts/SkBlitRow_opts_SSE4.h ('k') | src/opts/SkBlitRow_opts_SSE4_asm.S » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698