[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