[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