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

Side by Side Diff: tests/PMFloatTest.cpp

Issue 1029163002: Replace _mm_cvtps_epi32(x) with _mm_cvttps_epi32(_mm_add_ps(0.5f), x). (Closed) Base URL: https://skia.googlesource.com/skia@master
Patch Set: tweak test Created 5 years, 9 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/SkPMFloat_SSSE3.h ('k') | tests/Sk4xTest.cpp » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 #include "SkPMFloat.h" 1 #include "SkPMFloat.h"
2 #include "Test.h" 2 #include "Test.h"
3 3
4 DEF_TEST(SkPMFloat, r) { 4 DEF_TEST(SkPMFloat, r) {
5 // Test SkPMColor <-> SkPMFloat 5 // Test SkPMColor <-> SkPMFloat
6 SkPMColor c = SkPreMultiplyColor(0xFFCC9933); 6 SkPMColor c = SkPreMultiplyColor(0xFFCC9933);
7 SkPMFloat pmf(c); 7 SkPMFloat pmf(c);
8 REPORTER_ASSERT(r, SkScalarNearlyEqual(255.0f, pmf.a())); 8 REPORTER_ASSERT(r, SkScalarNearlyEqual(255.0f, pmf.a()));
9 REPORTER_ASSERT(r, SkScalarNearlyEqual(204.0f, pmf.r())); 9 REPORTER_ASSERT(r, SkScalarNearlyEqual(204.0f, pmf.r()));
10 REPORTER_ASSERT(r, SkScalarNearlyEqual(153.0f, pmf.g())); 10 REPORTER_ASSERT(r, SkScalarNearlyEqual(153.0f, pmf.g()));
11 REPORTER_ASSERT(r, SkScalarNearlyEqual( 51.0f, pmf.b())); 11 REPORTER_ASSERT(r, SkScalarNearlyEqual( 51.0f, pmf.b()));
12 REPORTER_ASSERT(r, c == pmf.get()); 12 REPORTER_ASSERT(r, c == pmf.get());
13 13
14 // Test rounding. (Don't bother testing .5... we don't care which way it go es.) 14 // Test rounding.
15 pmf = SkPMFloat(254.6f, 204.3f, 153.1f, 50.8f); 15 pmf = SkPMFloat(254.5f, 203.5f, 153.1f, 50.8f);
16 REPORTER_ASSERT(r, c == pmf.get()); 16 REPORTER_ASSERT(r, c == pmf.get());
17 17
18 // Test clamping. 18 // Test clamping.
19 SkPMFloat clamped(SkPMFloat(510.0f, 153.0f, 1.0f, -0.2f).clamped()); 19 SkPMFloat clamped(SkPMFloat(510.0f, 153.0f, 1.0f, -0.2f).clamped());
20 REPORTER_ASSERT(r, SkScalarNearlyEqual(255.0f, clamped.a())); 20 REPORTER_ASSERT(r, SkScalarNearlyEqual(255.0f, clamped.a()));
21 REPORTER_ASSERT(r, SkScalarNearlyEqual(153.0f, clamped.r())); 21 REPORTER_ASSERT(r, SkScalarNearlyEqual(153.0f, clamped.r()));
22 REPORTER_ASSERT(r, SkScalarNearlyEqual( 1.0f, clamped.g())); 22 REPORTER_ASSERT(r, SkScalarNearlyEqual( 1.0f, clamped.g()));
23 REPORTER_ASSERT(r, SkScalarNearlyEqual( 0.0f, clamped.b())); 23 REPORTER_ASSERT(r, SkScalarNearlyEqual( 0.0f, clamped.b()));
24 24
25 // Test SkPMFloat <-> Sk4f conversion. 25 // Test SkPMFloat <-> Sk4f conversion.
(...skipping 18 matching lines...) Expand all
44 SkPMFloat::To4PMColors(back, floats); 44 SkPMFloat::To4PMColors(back, floats);
45 for (int i = 0; i < 4; i++) { 45 for (int i = 0; i < 4; i++) {
46 REPORTER_ASSERT(r, back[i] == colors[i]); 46 REPORTER_ASSERT(r, back[i] == colors[i]);
47 } 47 }
48 48
49 SkPMFloat::ClampTo4PMColors(back, floats); 49 SkPMFloat::ClampTo4PMColors(back, floats);
50 for (int i = 0; i < 4; i++) { 50 for (int i = 0; i < 4; i++) {
51 REPORTER_ASSERT(r, back[i] == colors[i]); 51 REPORTER_ASSERT(r, back[i] == colors[i]);
52 } 52 }
53 } 53 }
OLDNEW
« no previous file with comments | « src/opts/SkPMFloat_SSSE3.h ('k') | tests/Sk4xTest.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698