[PATCH] D111451: [DenseMapInfo] Move hash_code implementation to Hashing.h

[Nikita Popov via Phabricator via llvm-commits]

nikic added a comment.

In D111451#3052300 <https://reviews.llvm.org/D111451#3052300>, @lattner wrote:

> nice, makes sense.  I'm surprised more `#include <string>` instances weren't needed in other downstream headers and .cpp files.

I landed the necessary (at least for my build) include fixups ahead of time in https://github.com/llvm/llvm-project/commit/cfb53d8e6d6383c9622331224acf0abb03cb1713. Interestingly no `<string>` includes in particular were needed.


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

https://reviews.llvm.org/D111451