[PATCH] D158250: [IR] Add more details to StructuralHash

Aiden Grossman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 21 23:27:59 PDT 2023


aidengrossman updated this revision to Diff 552229.
aidengrossman added a comment.

Update patch

- Switch to native hashing implementations for APInt/APFloat
- Switch from hashing type pointers to hashing some details about the type as the former is non-deterministic (which of course is frustratingly masked by my system having ASLR disabled). Left a TODO to refactor the type hashing into a more formal hash_value implementation within the class.
- Formatting changes.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D158250

Files:
  llvm/include/llvm/IR/StructuralHash.h
  llvm/lib/IR/Pass.cpp
  llvm/lib/IR/StructuralHash.cpp
  llvm/unittests/IR/StructuralHashTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D158250.552229.patch
Type: text/x-patch
Size: 12731 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20230822/5b4359cc/attachment.bin>


More information about the llvm-commits mailing list