[llvm] [LLVM][ADT] Convert llvm::hash_code to unsigned explicitly in DenseMapInfo (PR #77743)

David Blaikie via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 11 10:24:35 PST 2024


dwblaikie wrote:

Yeah, if `getHashValue` returns size_t, we'd have the same narrowing issue inside DenseMap itself.

But I think the right solution would be to do the same thing as is done for `unsigned long long` - maybe even just have the size_t specialization delegate to the unsigned long long implementation (maybe only if sizeo(size_t) > sizeof(unsigned)?)?

https://github.com/llvm/llvm-project/pull/77743


More information about the llvm-commits mailing list