Re: [PATCH 1/2] rs6000: Add support for SSE4.1 "ceil" intrinsics

[Bill Schmidt via Gcc-patches]

Hi Paul,

On 7/1/21 9:11 PM, Paul A. Clarke via Gcc-patches wrote:
2021-07-01  Paul A. Clarke  <p...@us.ibm.com>
gcc/ChangeLog:
        * config/rs6000/smmintrin.h (_mm_ceil_pd, _mm_ceil_ps,
        _mm_ceil_sd, _mm_ceil_ss): New.
---
  gcc/config/rs6000/smmintrin.h | 28 ++++++++++++++++++++++++++++
  1 file changed, 28 insertions(+)

diff --git a/gcc/config/rs6000/smmintrin.h b/gcc/config/rs6000/smmintrin.h
index fa17a8b2f478..0c0b0dd7c1e3 100644
--- a/gcc/config/rs6000/smmintrin.h
+++ b/gcc/config/rs6000/smmintrin.h
@@ -212,4 +212,32 @@ _mm_test_mix_ones_zeros (__m128i __A, __m128i __mask)
    return any_ones * any_zeros;
  }

+extern __inline __m128d __attribute__((__gnu_inline__, __always_inline__, 
__artificial__))

Usual fuss over line length, here and below.  Otherwise LGTM, can't 
approve but recommend approval with those changes.

Thanks,
Bill

+_mm_ceil_pd (__m128d __A)
+{
+  return (__m128d) vec_ceil ((__v2df) __A);
+}
+
+extern __inline __m128 __attribute__((__gnu_inline__, __always_inline__, 
__artificial__))
+_mm_ceil_ps (__m128 __A)
+{
+  return (__m128) vec_ceil ((__v4sf) __A);
+}
+
+extern __inline __m128d __attribute__((__gnu_inline__, __always_inline__, 
__artificial__))
+_mm_ceil_sd (__m128d __A, __m128d __B)
+{
+  __v2df r = vec_ceil ((__v2df) __B);
+  r[1] = ((__v2df) __A)[1];
+  return (__m128d) r;
+}
+
+extern __inline __m128 __attribute__((__gnu_inline__, __always_inline__, 
__artificial__))
+_mm_ceil_ss (__m128 __A, __m128 __B)
+{
+  __v4sf r = (__v4sf) __A;
+  r[0] = __builtin_ceil (((__v4sf) __B)[0]);
+  return r;
+}
+
  #endif