[PATCH] D120121: [StableHashing] Hash vregs with multiple defs

Kyungwoo Lee via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 18 09:05:54 PST 2022


kyulee added a comment.

Stable hash was designed to infer the machine instructions that are likely to be identical so that we can optimistically outline them.
I don't have a strong suggestion on how to interpret this in the presence of virtual registers (before RA).
But by taking all their defs into account, it appears the hash is likely to be more unique than before.
Is there any specific use case or reason to enhance this collision?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D120121



More information about the llvm-commits mailing list