From mboxrd@z Thu Jan 1 00:00:00 1970 From: Wu Fengguang Subject: [PATCH 2/6] bitops: compile time optimization for hweight_long(CONSTANT) Date: Fri, 15 Jan 2010 09:39:56 +0800 Message-ID: <20100115014422.405089087@intel.com> References: <20100115013954.311049665@intel.com> Cc: Al Viro , Jamie Lokier , Roland Dreier , Wu Fengguang To: Andrew Morton Return-path: CC: Christoph Hellwig Cc: LKML CC: Eric Paris CC: Nick Piggin CC: Andi Kleen CC: David Howells CC: Jonathan Corbet cc: Content-Disposition: inline; filename=constant-hweight32.patch Sender: linux-kernel-owner@vger.kernel.org List-Id: linux-fsdevel.vger.kernel.org This allows use of hweight_long() in BUILD_BUG_ON(). Suggested by Jamie. CC: Jamie Lokier CC: Roland Dreier Signed-off-by: Wu Fengguang --- include/linux/bitops.h | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) --- linux.orig/include/linux/bitops.h 2010-01-09 12:13:00.000000000 +0800 +++ linux/include/linux/bitops.h 2010-01-09 12:21:50.000000000 +0800 @@ -40,10 +40,14 @@ static __inline__ int get_count_order(un return order; } -static inline unsigned long hweight_long(unsigned long w) -{ - return sizeof(w) == 4 ? hweight32(w) : hweight64(w); -} +#define hweight_long(x) \ +( \ + __builtin_constant_p(x) ? \ + __builtin_popcountl(x) : \ + (sizeof(x) <= 4 ? \ + hweight32(x) : \ + hweight64(x)) \ +) /** * rol32 - rotate a 32-bit value left