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

Unified Diff: src/core/SkBitmapProcState.cpp

Issue 656913005: Revert of create shaderproc for nofilter-opaque-dx (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: Created 6 years, 2 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 | « bench/BitmapBench.cpp ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/core/SkBitmapProcState.cpp
diff --git a/src/core/SkBitmapProcState.cpp b/src/core/SkBitmapProcState.cpp
index b1b50b0c329af9eb727f5710353fb742b1c075d7..2a449d6d8fa8e7732732bd9046517eb2a2ddb57d 100644
--- a/src/core/SkBitmapProcState.cpp
+++ b/src/core/SkBitmapProcState.cpp
@@ -29,8 +29,6 @@
extern void SI8_opaque_D32_filter_DX_shaderproc_neon(const SkBitmapProcState&, int, int, uint32_t*, int);
extern void Clamp_SI8_opaque_D32_filter_DX_shaderproc_neon(const SkBitmapProcState&, int, int, uint32_t*, int);
#endif
-
-extern void Clamp_S32_opaque_D32_nofilter_DX_shaderproc(const SkBitmapProcState&, int, int, uint32_t*, int);
#define NAME_WRAP(x) x
#include "SkBitmapProcState_filter.h"
@@ -591,8 +589,6 @@
}
} else if (SK_ARM_NEON_WRAP(SI8_opaque_D32_filter_DX) == fSampleProc32 && clampClamp) {
fShaderProc32 = SK_ARM_NEON_WRAP(Clamp_SI8_opaque_D32_filter_DX_shaderproc);
- } else if (S32_opaque_D32_nofilter_DX == fSampleProc32 && clampClamp) {
- fShaderProc32 = Clamp_S32_opaque_D32_nofilter_DX_shaderproc;
}
if (NULL == fShaderProc32) {
@@ -1023,58 +1019,3 @@
return size;
}
-
-///////////////////////
-
-void Clamp_S32_opaque_D32_nofilter_DX_shaderproc(const SkBitmapProcState& s, int x, int y,
- SkPMColor* SK_RESTRICT dst, int count) {
- SkASSERT((s.fInvType & ~(SkMatrix::kTranslate_Mask |
- SkMatrix::kScale_Mask)) == 0);
-
- const unsigned maxX = s.fBitmap->width() - 1;
- SkFractionalInt fx;
- int dstY;
- {
- SkPoint pt;
- s.fInvProc(s.fInvMatrix, SkIntToScalar(x) + SK_ScalarHalf, SkIntToScalar(y) + SK_ScalarHalf,
- &pt);
- fx = SkScalarToFractionalInt(pt.fY);
- const unsigned maxY = s.fBitmap->height() - 1;
- dstY = SkClampMax(SkFractionalIntToInt(fx), maxY);
- fx = SkScalarToFractionalInt(pt.fX);
- }
-
- const SkPMColor* SK_RESTRICT src = s.fBitmap->getAddr32(0, dstY);
- const SkFractionalInt dx = s.fInvSxFractionalInt;
-
- // Check if we're safely inside [0...maxX] so no need to clamp each computed index.
- //
- if ((uint64_t)SkFractionalIntToInt(fx) <= maxX &&
- (uint64_t)SkFractionalIntToInt(fx + dx * (count - 1)) <= maxX)
- {
- int count4 = count >> 2;
- for (int i = 0; i < count4; ++i) {
- SkPMColor src0 = src[SkFractionalIntToInt(fx)]; fx += dx;
- SkPMColor src1 = src[SkFractionalIntToInt(fx)]; fx += dx;
- SkPMColor src2 = src[SkFractionalIntToInt(fx)]; fx += dx;
- SkPMColor src3 = src[SkFractionalIntToInt(fx)]; fx += dx;
- dst[0] = src0;
- dst[1] = src1;
- dst[2] = src2;
- dst[3] = src3;
- dst += 4;
- }
- for (int i = (count4 << 2); i < count; ++i) {
- unsigned index = SkFractionalIntToInt(fx);
- SkASSERT(index <= maxX);
- *dst++ = src[index];
- fx += dx;
- }
- } else {
- for (int i = 0; i < count; ++i) {
- dst[i] = src[SkClampMax(SkFractionalIntToInt(fx), maxX)];
- fx += dx;
- }
- }
-}
-
« no previous file with comments | « bench/BitmapBench.cpp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698