[all-commits] [llvm/llvm-project] f991eb: [Support] Add llvm::xxh3_128bits (#95863)

Brendan Duke via All-commits all-commits at lists.llvm.org
Wed Jun 19 12:25:16 PDT 2024


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: f991ebbb465301a1382d6ce15a346b4edc3cfce2
      https://github.com/llvm/llvm-project/commit/f991ebbb465301a1382d6ce15a346b4edc3cfce2
  Author: Brendan Duke <brendanw.duke at gmail.com>
  Date:   2024-06-19 (Wed, 19 Jun 2024)

  Changed paths:
    M llvm/include/llvm/Support/xxhash.h
    M llvm/lib/Support/xxhash.cpp
    M llvm/unittests/Support/xxhashTest.cpp

  Log Message:
  -----------
  [Support] Add llvm::xxh3_128bits (#95863)

Add a 128-bit xxhash function, following the existing
`llvm::xxh3_64bits` and `llvm::xxHash` implementations. Previously,
48e93f57f1ee914ca29aa31bf2ccd916565a3610 added support for
`llvm::xxh3_64bits`, which closely follows the upstream implementation
at https://github.com/Cyan4973/xxHash, with simplifications from Devin
Hussey's xxhash-clean.
However, it is desirable to have a larger 128-bit hash key for use cases
such as filesystem checksums where chance of collision needs to be
negligible.

So to that end this also ports over the 128-bit xxh3_128bits as
`llvm::xxh3_128bits`.

Testing:
- Add a test based on xsum_sanity_check.c in upstream xxhash.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications


More information about the All-commits mailing list