On 5/3/23 10:17, Alex Bennée wrote:
Now we no longer have users for qemu_xxhash7 we can drop an additional
multiply and rol and make qemu_xxhash6 the implementation. Adjust the
smaller hash functions accordingly.
Message-Id: <[email protected]>
Reviewed-by: Philippe Mathieu-Daudé <[email protected]>
Reviewed-by: Stefan Hajnoczi <[email protected]>
Signed-off-by: Alex Bennée <[email protected]>
---
include/qemu/xxhash.h | 17 ++++-------------
1 file changed, 4 insertions(+), 13 deletions(-)
Oh, well, I noticed during this review that we're not hashing cs_base, so probably
expanding this to xxhash8 more proper.
cs_base is used for all kinds of important things, such as AArch64 flags expansion, so we
might get better hashing performance by including it.
r~
