[PATCH] D113641: [llvm] Add a SFINAE template parameter to DenseMapInfo

[Chris Lattner via Phabricator via cfe-commits]

lattner accepted this revision.
lattner added a comment.
Herald added a subscriber: sdasgup3.

Nice, I'm very excited about this - this will allow a lot of cleanups.  Thank you for doing this!



================
Comment at: llvm/include/llvm/ADT/Hashing.h:59
 namespace llvm {
-template <typename T> struct DenseMapInfo;
 
----------------
Is there a way to keep the forward declarations references here instead of #include?  DenseMapInfo.h pulls in a ton of stuff including <utility> and <tuple>


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D113641