IncludeGuardian added inline comments.
================
Comment at: llvm/include/llvm/ADT/DenseMapInfo.h:321
- static bool isEqual(const Variant &LHS, const Variant &RHS) {
- return LHS == RHS;
- }
----------------
Regardless of this change, I think the equality here needs to be fixed in the
future to delegate to `DenseMapInfo` for checking equality of the contents of
the `std::variant`
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D150997/new/
https://reviews.llvm.org/D150997
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
