It seems to me that shifting the variable 'a' at the end of the
functions is of no use. Maybe there is something I don't see here, in
which case I apologize. :)
--
Zaga <[email protected]>
What can change the nature of a man?
>From 772c6517400e45c6e07c7f0fd733d9cbd68c65da Mon Sep 17 00:00:00 2001
From: Miroslav Zagorac <[email protected]>
Date: Sun, 8 Mar 2020 16:32:20 +0100
Subject: [PATCH] CLEANUP: remove unused code in 'my_ffsl/my_flsl' functions
Shifting the variable 'a' one bit to the right has no effect on the
result of the functions.
---
include/common/standard.h | 2 --
1 file changed, 2 deletions(-)
diff --git a/include/common/standard.h b/include/common/standard.h
index d8bccbafb..103a07e5f 100644
--- a/include/common/standard.h
+++ b/include/common/standard.h
@@ -902,7 +902,6 @@ static inline unsigned int my_ffsl(unsigned long a)
cnt += 2;
}
if (!(a & 0x1)) {
- a >>= 1;
cnt += 1;
}
#endif /* x86_64 */
@@ -946,7 +945,6 @@ static inline unsigned int my_flsl(unsigned long a)
cnt += 2;
}
if (a & 0x2) {
- a >>= 1;
cnt += 1;
}
#endif /* x86_64 */
--
2.20.1
