use a full 8-bit table for popcount lookup

spaskalev · Jun 10, 2023 · 8332066 · 8332066
1 parent 2c6deda
commit 8332066
Showing 1 changed file with 12 additions and 3 deletions.
diff --git a/buddy_alloc.h b/buddy_alloc.h
@@ -1795,10 +1795,19 @@ static void bitset_debug(FILE *stream, unsigned char *bitset, size_t length) {
  Bits
 */
 
-static const unsigned char popcount_lookup[16] = {0, 1, 1, 2, 1, 2, 2, 3, 1, 2, 2, 3, 2, 3, 3, 4};
+static const unsigned char popcount_lookup[256] = {
+    0,1,1,2,1,2,2,3,1,2,2,3,2,3,3,4,1,2,2,3,2,3,3,4,2,3,3,4,3,4,4,5,
+    1,2,2,3,2,3,3,4,2,3,3,4,3,4,4,5,2,3,3,4,3,4,4,5,3,4,4,5,4,5,5,6,
+    1,2,2,3,2,3,3,4,2,3,3,4,3,4,4,5,2,3,3,4,3,4,4,5,3,4,4,5,4,5,5,6,
+    2,3,3,4,3,4,4,5,3,4,4,5,4,5,5,6,3,4,4,5,4,5,5,6,4,5,5,6,5,6,6,7,
+    1,2,2,3,2,3,3,4,2,3,3,4,3,4,4,5,2,3,3,4,3,4,4,5,3,4,4,5,4,5,5,6,
+    2,3,3,4,3,4,4,5,3,4,4,5,4,5,5,6,3,4,4,5,4,5,5,6,4,5,5,6,5,6,6,7,
+    2,3,3,4,3,4,4,5,3,4,4,5,4,5,5,6,3,4,4,5,4,5,5,6,4,5,5,6,5,6,6,7,
+    3,4,4,5,4,5,5,6,4,5,5,6,5,6,6,7,4,5,5,6,5,6,6,7,5,6,6,7,6,7,7,8
+};
 
-static unsigned int popcount_byte(unsigned char b) {
-    return popcount_lookup[b & 15] + popcount_lookup[b >> 4];
+static inline unsigned int popcount_byte(unsigned char b) {
+    return popcount_lookup[b];
 }
 
 /* Returns the highest set bit position for the given value. Returns zero for zero. */