From: VMware, Inc <>
Date: Thu, 17 Jun 2010 22:06:10 +0000 (-0700)
Subject: bit functions: a little nicer implementation
X-Git-Tag: 2010.06.16-268169~55
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=f2c698be6bac5ddfb4b2d05dc770d50ae2a6cf9c;p=thirdparty%2Fopen-vm-tools.git

bit functions: a little nicer implementation

Signed-off-by: Marcelo Vanzin <mvanzin@vmware.com>
---

diff --git a/open-vm-tools/lib/include/vm_basic_asm.h b/open-vm-tools/lib/include/vm_basic_asm.h
index 6b395c2eb..a117ffbf0 100644
--- a/open-vm-tools/lib/include/vm_basic_asm.h
+++ b/open-vm-tools/lib/include/vm_basic_asm.h
@@ -336,19 +336,17 @@ mssb64_0(const uint64 value)
 static INLINE int
 lssb32_0(uint32 value)
 {
+#if defined(USE_ARCH_X86_CUSTOM)
    if (UNLIKELY(value == 0)) {
       return -1;
    } else {
       int pos;
-
-#if defined(USE_ARCH_X86_CUSTOM)
       __asm__ __volatile__("bsfl %1, %0\n" : "=r" (pos) : "rm" (value) : "cc");
-#else
-      pos =  __builtin_ffs(value) - 1;
-#endif
-
       return pos;
    }
+#else
+   return __builtin_ffs(value) - 1;
+#endif
 }
 
 static INLINE int
@@ -372,12 +370,12 @@ mssb32_0(uint32 value)
 static INLINE int
 lssb64_0(const uint64 value)
 {
+#if defined(USE_ARCH_X86_CUSTOM)
    if (UNLIKELY(value == 0)) {
       return -1;
    } else {
       intptr_t pos;
 
-#if defined(USE_ARCH_X86_CUSTOM)
 #if defined(VM_X86_64)
       __asm__ __volatile__("bsf %1, %0\n" : "=r" (pos) : "rm" (value) : "cc");
 #else
@@ -390,12 +388,12 @@ lssb64_0(const uint64 value)
          }
       }
 #endif
-#else
-      pos =  __builtin_ffsll(value) - 1;
-#endif
 
       return pos;
    }
+#else
+   return __builtin_ffsll(value) - 1;
+#endif
 }
 
 static INLINE int