[llvm] [StructuralHash] Refactor (PR #112621)
Nikita Popov via llvm-commits
llvm-commits at lists.llvm.org
Tue Nov 5 05:20:11 PST 2024
nikic wrote:
> > Why the use of stable_hash? This hash is not expected to be stable, e.g. because we may start hashing additional things into it.
>
> As posted in https://discourse.llvm.org/t/rfc-global-function-merging/82608, I plan to make this structural hash stable (in the following PR). If it is preferable to maintain the existing structural hash implementation as is, I am open to creating a separate data structure or extending it through a type hierarchy.
Just to check, how "stable" is stable here? Is this just stable for a given LLVM build, or will it have to be stable across versions?
https://github.com/llvm/llvm-project/pull/112621
More information about the llvm-commits
mailing list