Dont do popcount in sse2 path either

Author: byroot

ext/json/ext/generator/generator.c

@@ -115,7 +115,7 @@ typedef struct _search_state {
 #ifdef ENABLE_SIMD
     const char *chunk_base;
     const char *chunk_end;
-    uint8_t has_matches;
+    bool has_matches;
 
 #ifdef HAVE_SIMD_NEON
     uint64_t matches_mask;
@@ -223,11 +223,8 @@ static inline FORCE_INLINE void escape_UTF8_char_basic(search_state *search)
  */
 static inline void convert_UTF8_to_JSON(search_state *search)
 {
-    unsigned char num_chars = 0;
-    while ((num_chars = search_escape_basic_impl(search))) {
-        do {
-            escape_UTF8_char_basic(search);
-        } while (--num_chars);
+    while (search_escape_basic_impl(search)) {
+        escape_UTF8_char_basic(search);
     }
 }
 
@@ -510,10 +507,6 @@ static inline TARGET_SSE2 FORCE_INLINE unsigned char search_escape_basic_sse2(se
             continue;
         }
 
-        if (popcount32(needs_escape_mask) >= sizeof(__m128i)/2) {
-            return sizeof(__m128i);
-        }
-
         search->has_matches = 1;
         search->matches_mask = needs_escape_mask;
         search->chunk_base = search->ptr;
@@ -539,10 +532,6 @@ static inline TARGET_SSE2 FORCE_INLINE unsigned char search_escape_basic_sse2(se
             return 0;
         }
 
-        if (popcount32(needs_escape_mask) >= sizeof(__m128i)/2) {
-            return remaining;
-        }
-
         search->has_matches = 1;
         search->matches_mask = needs_escape_mask;
         search->chunk_base = search->ptr;

ext/json/ext/generator/simd.h

@@ -48,20 +48,6 @@ static inline int trailing_zeros(int input) {
   #endif
 }
 
-uint32_t popcount32(uint32_t x) {
-  #if defined(__GNUC__) || defined(__clang__)
-    return __builtin_popcount(x);
-  #elif defined(__ARM_NEON)
-      #include <arm_neon.h>
-      return vaddv_u8(vcnt_u8(vcreate_u8((uint64_t)x))) & 0xFF;
-  #else
-      x = x - ((x >> 1) & 0x55555555);
-      x = (x & 0x33333333) + ((x >> 2) & 0x33333333);
-      x = (x + (x >> 4)) & 0x0F0F0F0F;
-      return (x * 0x01010101) >> 24;
-  #endif
-}
-
 #define SIMD_MINIMUM_THRESHOLD 6
 
 #if defined(__ARM_NEON) || defined(__ARM_NEON__) || defined(__aarch64__) || defined(_M_ARM64)