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

[Aiden Grossman via Phabricator via llvm-commits]

aidengrossman updated this revision to Diff 552146.
aidengrossman marked 3 inline comments as done.
aidengrossman added a comment.

Address reviewer feedback

- Early exit where pointed out.
- Switch to hashing operands for the most part (except for in the case of cmp)
- Add a couple additional operands to hash, particularly integer constants and arguments.
- Rebase against upstream to include changes to prevent test regressions on 32-bit arches.


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.552146.patch
Type: text/x-patch
Size: 12554 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20230821/587aff50/attachment.bin>