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

River Riddle via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 10 19:15:20 PST 2021


rriddle added a comment.

Given the new template parameter, I needed to update the forward declarations. Some of them already had DenseMapInfo from an include, so I just dropped them. Some didn't, so I opted to just add an include for DenseMapInfo (seemed small enough). I could avoid the includes by adding an explicit `void` for the second template parameter. Let me know if there is a preference.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D113641



More information about the llvm-commits mailing list