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

Unified Diff: src/opts/SkBitmapProcState_opts_arm.cpp

Issue 23796005: remove fConvolutionProcs from State, and just use it locally (Closed) Base URL: https://skia.googlecode.com/svn/trunk
Patch Set: Created 7 years, 3 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « src/core/SkConvolver.cpp ('k') | src/opts/opts_check_SSE2.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/opts/SkBitmapProcState_opts_arm.cpp
diff --git a/src/opts/SkBitmapProcState_opts_arm.cpp b/src/opts/SkBitmapProcState_opts_arm.cpp
index a9b44daa890489d319355619d169294b54297157..3a3cb8567c9525a0719fdfa99493a63bcfad6ff6 100644
--- a/src/opts/SkBitmapProcState_opts_arm.cpp
+++ b/src/opts/SkBitmapProcState_opts_arm.cpp
@@ -401,10 +401,10 @@ void applySIMDPadding_arm(SkConvolutionFilter1D *filter) {
}
}
-void SkBitmapProcState::platformConvolutionProcs() {
+void SkBitmapProcState::platformConvolutionProcs(SkConvolutionProcs* procs) {
if (sk_cpu_arm_has_neon()) {
- fConvolutionProcs->fExtraHorizontalReads = 3;
- fConvolutionProcs->fConvolveVertically = &convolveVertically_arm;
+ procs->fExtraHorizontalReads = 3;
+ procs->fConvolveVertically = &convolveVertically_arm;
// next line is commented out because the four-row convolution function above is
// just a no-op. Please see the comment above its definition, and the SSE implementation
@@ -412,11 +412,11 @@ void SkBitmapProcState::platformConvolutionProcs() {
// leaving it as NULL will just cause the convolution system to not attempt
// to operate on four rows at once, which is correct but not performance-optimal.
- // fConvolutionProcs->fConvolve4RowsHorizontally = &convolve4RowsHorizontally_arm;
+ // procs->fConvolve4RowsHorizontally = &convolve4RowsHorizontally_arm;
- fConvolutionProcs->fConvolve4RowsHorizontally = NULL;
+ procs->fConvolve4RowsHorizontally = NULL;
- fConvolutionProcs->fConvolveHorizontally = &convolveHorizontally_arm;
- fConvolutionProcs->fApplySIMDPadding = &applySIMDPadding_arm;
+ procs->fConvolveHorizontally = &convolveHorizontally_arm;
+ procs->fApplySIMDPadding = &applySIMDPadding_arm;
}
}
« no previous file with comments | « src/core/SkConvolver.cpp ('k') | src/opts/opts_check_SSE2.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698