[PATCH] D158217: [NFCi][MergeFunctions] Consolidate Hashing Functions

Arthur Eubanks via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 17 15:16:05 PDT 2023


aeubanks added a comment.

I think it's important to add comments in StructuralHash that different users of it may want different behavior, so in the future if that's desired we should be able to add flags to tune the behavior of the hashing

but otherwise this seems fine


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D158217



More information about the llvm-commits mailing list