gallium/util: Don't use __builtin_clrsb in util_last_bit().
authorMatt Turner <mattst88@gmail.com>
Tue, 3 Feb 2015 01:26:49 +0000 (17:26 -0800)
committerMatt Turner <mattst88@gmail.com>
Tue, 3 Feb 2015 20:25:14 +0000 (12:25 -0800)
Unclear circumstances lead to undefined symbols on x86.

Bugzilla: https://bugs.gentoo.org/show_bug.cgi?id=536916
Cc: mesa-stable@lists.freedesktop.org
Reviewed-by: Ilia Mirkin <imirkin@alum.mit.edu>
src/gallium/auxiliary/util/u_math.h

index 19c7343b171dfc96130b0df850acc137ab11e24f..ca6d0f1dfc845335c13653605920332ecbd67da4 100644 (file)
@@ -561,14 +561,10 @@ util_last_bit(unsigned u)
 static INLINE unsigned
 util_last_bit_signed(int i)
 {
-#if defined(__GNUC__) && ((__GNUC__ * 100 + __GNUC_MINOR__) >= 407) && !defined(__INTEL_COMPILER)
-   return 31 - __builtin_clrsb(i);
-#else
    if (i >= 0)
       return util_last_bit(i);
    else
       return util_last_bit(~(unsigned)i);
-#endif
 }
 
 /* Destructively loop over all of the bits in a mask as in: