[PATCH] D121510: [Support] Introduce the BLAKE3 hashing function implementation

Argyrios Kyrtzidis via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Mar 12 21:37:20 PST 2022


akyrtzi marked an inline comment as done.
akyrtzi added inline comments.


================
Comment at: llvm/include/llvm/Support/BLAKE3.h:78
+    Hasher.update(Data);
+    return Hasher.final();
+  }
----------------
MaskRay wrote:
> Otherwise we can't use `BLAKE3::hash<16>` to replace md5.
> 
> 
Thank you for catching this! 🙇🏻‍♂️


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D121510/new/

https://reviews.llvm.org/D121510



More information about the llvm-commits mailing list