src/opts/SkOpts_avx.cpp - Issue 2087343002: Add stub for avx.

Side by Side Diff

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

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

Issue 2087343002: Add stub for avx. (Closed) Base URL: https://skia.googlesource.com/skia.git@master

Patch Set: Sync Created 4 years, 6 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:

OLD	NEW
1 /*	1 /*

2 * Copyright 2016 Google Inc.	2 * Copyright 2016 Google Inc.

3 *	3 *

4 * Use of this source code is governed by a BSD-style license that can be	4 * Use of this source code is governed by a BSD-style license that can be

5 * found in the LICENSE file.	5 * found in the LICENSE file.

6 */	6 */

7	7

8 #include "SkSwizzle.h"

9

10 #include "SkOpts.h"	8 #include "SkOpts.h"

11	9
	mtklein_C 2016/06/22 20:33:41 Looks like you should be including <immintrin.h> e Looks like you should be including <immintrin.h> explicitly in SkBlend_opts.h. SkNx doesn't necessarily include it for you, even on x86. We have a build mode where SkNx.h does not use SSE intrinsics on x86. We like this bot as a control. SkOpts_ssse3.cpp and SkOpts_sse41.cpp have been getting away with it because they include other headers first that include <immintrin.h>. I also don't mind if we just have each of SkOpts_{ssse3,sse41,avx} include <immintrin.h>.
12 void SkSwapRB(uint32_t* dest, const uint32_t* src, int count) {	10 #define SK_OPTS_NS sk_avx

13 SkOpts::RGBA_to_BGRA(dest, src, count);	11 #include "SkBlend_opts.h"

	12

	13 namespace SkOpts {

	14 void Init_avx() { }

14 }	15 }

OLD	NEW

« no previous file with comments | « src/core/SkOpts.cpp ('k') | no next file » | no next file with comments »