Add Sk4f_RoundToInt

src/core/SkNx.h

 /*
  * Copyright 2015 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #ifndef SkNx_DEFINED
 #define SkNx_DEFINED
 
@@ skipping 281 matching lines @@
 typedef SkNx<8,  uint16_t> Sk8h;
 typedef SkNx<16, uint16_t> Sk16h;
 
 typedef SkNx<4,       int> Sk4i;
 
 // Include platform specific specializations if available.
 #if !defined(SKNX_NO_SIMD) && SK_CPU_SSE_LEVEL >= SK_CPU_SSE_LEVEL_SSE2
     #include "../opts/SkNx_sse.h"
 #elif !defined(SKNX_NO_SIMD) && defined(SK_ARM_HAS_NEON)
     #include "../opts/SkNx_neon.h"
+#else
+
+SI Sk4i Sk4f_RoundToInt(const Sk4f& x) {
+    return { sk_float_round2int(x[0]),

[mtklein, 2016/07/11 14:05:03]

If we can't implement this in terms of things from <math.h>  (e.g. lrintf) can
you leave a note about why we need to use Skia's homebrew instead?

[msarett, 2016/07/12 15:42:43]

lrintf() seems fine to me.

+             sk_float_round2int(x[1]),
+             sk_float_round2int(x[2]),
+             sk_float_round2int(x[3]), };
+}
+
+SI uint32_t Sk4f_RoundToByte(const Sk4f& x) {

[mtklein, 2016/07/11 14:05:03]

Why don't we simplify this by adding just one new call,

   Sk4i Sk4f_round(Sk4f);

and then make sure SkNx_cast<uint8_t>(Sk4i) is implemented well to handle the
rest of the trip down to bytes?

   SkNx_cast<uint8_t>(Sk4f_round(...)).store(&dst)

[msarett, 2016/07/12 15:42:43]

SGTM

+    return sk_float_round2int(x[0]) <<  0
+         | sk_float_round2int(x[1]) <<  8
+         | sk_float_round2int(x[2]) << 16
+         | sk_float_round2int(x[3]) << 24;
+}
+
 #endif
 
 SI void Sk4f_ToBytes(uint8_t p[16], const Sk4f& a, const Sk4f& b, const Sk4f& c, const Sk4f& d) {
     SkNx_cast<uint8_t>(SkNx_join(SkNx_join(a,b), SkNx_join(c,d))).store(p);
 }
 
 #undef SI
 
 #endif//SkNx_DEFINED