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

[Sean Silva via Phabricator via cfe-commits]

silvas accepted this revision.
silvas added a comment.
This revision is now accepted and ready to land.

Nice :) This looks pretty straightforward, but given how core this is, wait for another approver or two.



================
Comment at: llvm/include/llvm/ADT/DenseMapInfo.h:42
 
-template<typename T>
-struct DenseMapInfo {
+template <typename T, typename Enable = void> struct DenseMapInfo {
   //static inline T getEmptyKey();
----------------
bondhugula wrote:
> A code comment here?
+1. Is there a doc that needs to be updated too? We should update the source of truth is for "here's how to use DenseMapInfo", if such documentation even exists :)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D113641