[PATCH] D113461: [ADT] Add unit test for EquivalanceClasses comparator

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 8 22:50:19 PST 2021


dblaikie added inline comments.


================
Comment at: llvm/unittests/ADT/EquivalenceClassesTest.cpp:100-101
+TEST(EquivalenceClassesTest, Comparator) {
+  // This test is identical to MultipleSets, but uses `TestStruct` with a
+  // comparator instead of plain ints.
+
----------------
Could use a gunit type parameterized test for this, to avoid the duplication?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D113461



More information about the llvm-commits mailing list