tejohnson marked an inline comment as done.
tejohnson added inline comments.
================
Comment at: llvm/lib/Transforms/Instrumentation/PGOInstrumentation.cpp:1247
+ uint32_t Column) {
+ return hash_combine(Function, LineOffset, Column);
+}
----------------
MaskRay wrote:
> MaskRay wrote:
> > You may use BLAKE3 instead of MD5. BLAKE3 is much faster than LLVM's slow
> > MD5 implementation.
> llvm/include/llvm/Support/xxhash.h is also a good choice.
Thanks for the tip! I used BLAKE3.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D128142/new/
https://reviews.llvm.org/D128142
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
