On 04/07/2016 10:32 AM, Emilio G. Cota wrote:
> +#include <stdint.h>
Should already be done by <qemu/osdep.h>
> +/*
> + * Note : although _rotl exists for minGW (GCC under windows), performance
> + * seems poor.
> + */
> +#if defined(_MSC_VER)
> +# define XXH_rotl32(x, r) _rotl(x, r)
> +#else
> +# define XXH_rotl32(x, r) ((x << r) | (x >> (32 - r)))
> +#endif
Please use rol32 from <qemu/bitops.h>
> +static inline
> +uint32_t tb_hash_func5(uint64_t a0, uint64_t b0, uint32_t e, int seed)
> +{
Is there really any point in passing in "seed" anymore?
We certainly don't want different values passed in...
r~
