acl_hash: fix integer conversion warning

Avoid integer conversion altogether by using a more efficient
implementation which compiles to branchless code with a rotate
instruction, assuming the number of bits to rotate by is known
at compile time so the compiler may eliminate the assert().

Signed-off-by: Peter Colberg <peter.colberg@intel.com>

Author: pcolberg

lib/acl_hash/src/acl_hash.c

@@ -213,11 +213,11 @@ static void l_close(acl_hash_sha1_context_t *c, char *digest_buf) {
   digest_buf[40] = 0;
 }
 
-static uint32_t l_leftrotate(uint32_t v, unsigned bits) {
-  uint64_t both = ((uint64_t)v) << bits;
-  uint32_t hi = both >> 32;
-  uint32_t lo = both & 0xffffffff;
-  return hi | lo;
+// See Safe, Efficient, and Portable Rotate in C/C++ by John Regehr.
+// https://blog.regehr.org/archives/1063
+static uint32_t l_leftrotate(uint32_t v, uint32_t bits) {
+  assert(bits < 32);
+  return (v << bits) | (v >> ((-bits) & 31));
 }
 
 static uint32_t l_read_bigendian_i32(const unsigned char *buf) {