[PATCH] D131486: [llvm][ADT] Allow using structured bindings with `llvm::enumerate`

Jakub Kuderski via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 9 08:02:35 PDT 2022


kuhar added inline comments.


================
Comment at: llvm/unittests/ADT/STLExtrasTest.cpp:83-86
+  ASSERT_EQ(3u, foobar.size());
+  EXPECT_EQ(4, foobar[0]);
+  EXPECT_EQ(5, foobar[1]);
+  EXPECT_EQ(6, foobar[2]);
----------------
We can simplify this sequence of checks to `EXPECT_THAT(foobar, ElementsAre(4, 5, 6));`


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

https://reviews.llvm.org/D131486



More information about the llvm-commits mailing list