From: Richard Henderson <rth@twiddle.net>
Date: Thu, 14 Feb 2013 01:47:40 +0000 (-0800)
Subject: target-i386: Inline bitops_flsl
X-Git-Tag: v1.5.0-rc0~681
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=14e534265aad0796690a5aea429f5bd2877ccff0;p=thirdparty%2Fqemu.git

target-i386: Inline bitops_flsl

Use clz32 directly.  Which makes slightly more sense given
that the input is type "int" and not type "long".

Signed-off-by: Richard Henderson <rth@twiddle.net>
Reviewed-by: Eric Blake <eblake@redhat.com>
Signed-off-by: Blue Swirl <blauwirbel@gmail.com>
---

diff --git a/target-i386/topology.h b/target-i386/topology.h
index 24ed525453e..07a6c5fb559 100644
--- a/target-i386/topology.h
+++ b/target-i386/topology.h
@@ -52,10 +52,8 @@ typedef uint32_t apic_id_t;
 static unsigned apicid_bitwidth_for_count(unsigned count)
 {
     g_assert(count >= 1);
-    if (count == 1) {
-        return 0;
-    }
-    return bitops_flsl(count - 1) + 1;
+    count -= 1;
+    return count ? 32 - clz32(count) : 0;
 }
 
 /* Bit width of the SMT_ID (thread ID) field on the APIC ID