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

Unified Diff: src/core/SkRasterPipelineBlitter.cpp

Issue 2256023002: Flush denorm half floats to zero. (Closed) Base URL: https://skia.googlesource.com/skia.git@master
Patch Set: names Created 4 years, 4 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
Index: src/core/SkRasterPipelineBlitter.cpp
diff --git a/src/core/SkRasterPipelineBlitter.cpp b/src/core/SkRasterPipelineBlitter.cpp
index 0fae6dd5a29d3dea34837eb4723950574cefb9ee..427bbfa152cdc7ebe2077b8bba327dff21cae6bb 100644
--- a/src/core/SkRasterPipelineBlitter.cpp
+++ b/src/core/SkRasterPipelineBlitter.cpp
@@ -196,17 +196,17 @@ SK_RASTER_STAGE(load_d_f16) {
Sk4h rh, gh, bh, ah;
Sk4h_load4(ptr, &rh, &gh, &bh, &ah);
- dr = SkHalfToFloat_finite(rh);
- dg = SkHalfToFloat_finite(gh);
- db = SkHalfToFloat_finite(bh);
- da = SkHalfToFloat_finite(ah);
+ dr = SkHalfToFloat_ordinary(rh);
+ dg = SkHalfToFloat_ordinary(gh);
+ db = SkHalfToFloat_ordinary(bh);
+ da = SkHalfToFloat_ordinary(ah);
}
// Load 1 F16 pixel.
SK_RASTER_STAGE(load_d_f16_1) {
auto ptr = (const uint64_t*)ctx + x;
- auto p0 = SkHalfToFloat_finite(ptr[0]);
+ auto p0 = SkHalfToFloat_ordinary(ptr[0]);
dr = { p0[0],0,0,0 };
dg = { p0[1],0,0,0 };
db = { p0[2],0,0,0 };
@@ -217,15 +217,15 @@ SK_RASTER_STAGE(load_d_f16_1) {
SK_RASTER_STAGE(store_f16) {
auto ptr = (uint64_t*)ctx + x;
- Sk4h_store4(ptr, SkFloatToHalf_finite(r), SkFloatToHalf_finite(g),
- SkFloatToHalf_finite(b), SkFloatToHalf_finite(a));
+ Sk4h_store4(ptr, SkFloatToHalf_ordinary(r), SkFloatToHalf_ordinary(g),
+ SkFloatToHalf_ordinary(b), SkFloatToHalf_ordinary(a));
}
// Store 1 F16 pixel.
SK_RASTER_STAGE(store_f16_1) {
auto ptr = (uint64_t*)ctx + x;
- SkFloatToHalf_finite({r[0], g[0], b[0], a[0]}).store(ptr);
+ SkFloatToHalf_ordinary({r[0], g[0], b[0], a[0]}).store(ptr);
}
// Load 4 8-bit sRGB pixels from SkPMColor order to RGBA.

Powered by Google App Engine
This is Rietveld 408576698