commit 629f25a7ef229d97e22530eae043882b20cd8d8c
author mtklein <mtklein@chromium.org> Mon Feb 08 05:54:38 2016 -0800
committer Commit bot <commit-bot@chromium.org> Mon Feb 08 05:54:38 2016 -0800
tree f45890121140e9750c38e0f26964e9a45bb06665
parent a58ec2162c0a1e0c934d4ce485019303c51349a4

fix float <---> uint16_t conversion, with Mike's tests.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1679713002

patch from issue 1679713002 at patchset 1 (http://crrev.com/1679713002#ps1)
CQ_EXTRA_TRYBOTS=client.skia:Test-Ubuntu-GCC-GCE-CPU-AVX2-x86_64-Release-SKNX_NO_SIMD-Trybot

Review URL: https://codereview.chromium.org/1676893002

tests/SkNxTest.cpp

diff --git a/tests/SkNxTest.cpp b/tests/SkNxTest.cpp
index fcd113d..49e920f 100644
--- a/tests/SkNxTest.cpp
+++ b/tests/SkNxTest.cpp
@@ -224,3 +224,50 @@
     REPORTER_ASSERT(r, fs.kth<2>() == 2.0f);
     REPORTER_ASSERT(r, fs.kth<3>() == 4.0f);
 }
+
+#include "SkRandom.h"
+
+static void dump(const Sk4f& f4, const Sk4h& h4) {
+    SkDebugf("%g %g %g %g --> %d %d %d %d\n",
+             f4.kth<0>(), f4.kth<1>(), f4.kth<2>(), f4.kth<3>(),
+             h4.kth<0>(), h4.kth<1>(), h4.kth<2>(), h4.kth<3>());
+}
+
+DEF_TEST(SkNx_u16_float, r) {
+    {
+        // u16 --> float
+        auto h4 = Sk4h(15, 17, 257, 65535);
+        auto f4 = SkNx_cast<float>(h4);
+        dump(f4, h4);
+        REPORTER_ASSERT(r, f4.kth<0>() == 15.0f);
+        REPORTER_ASSERT(r, f4.kth<1>() == 17.0f);
+        REPORTER_ASSERT(r, f4.kth<2>() == 257.0f);
+        REPORTER_ASSERT(r, f4.kth<3>() == 65535.0f);
+    }
+    {
+        // float -> u16
+        auto f4 = Sk4f(15, 17, 257, 65535);
+        auto h4 = SkNx_cast<uint16_t>(f4);
+        dump(f4, h4);
+        REPORTER_ASSERT(r, h4.kth<0>() == 15);
+        REPORTER_ASSERT(r, h4.kth<1>() == 17);
+        REPORTER_ASSERT(r, h4.kth<2>() == 257);
+        REPORTER_ASSERT(r, h4.kth<3>() == 65535);
+    }
+
+    // starting with any u16 value, we should be able to have a perfect round-trip in/out of floats
+    //
+    SkRandom rand;
+    for (int i = 0; i < 0; ++i) {
+        const uint16_t s16[4] {
+            (uint16_t)rand.nextU16(), (uint16_t)rand.nextU16(),
+            (uint16_t)rand.nextU16(), (uint16_t)rand.nextU16(),
+        };
+        auto u4_0 = Sk4h::Load(s16);
+        auto f4 = SkNx_cast<float>(u4_0);
+        auto u4_1 = SkNx_cast<uint16_t>(f4);
+        uint16_t d16[4];
+        u4_1.store(d16);
+        REPORTER_ASSERT(r, !memcmp(s16, d16, sizeof(s16)));
+    }
+}