Dont do popcount in sse2 path either

Author: byroot

ext/json/ext/generator/generator.c

@@ -510,10 +510,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 +535,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)