[PATCH] D122079: [ADT] add initializer list specialization for is_contained

Jakub Kuderski via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 29 10:27:19 PDT 2022


kuhar added inline comments.


================
Comment at: llvm/unittests/ADT/STLExtrasTest.cpp:985-989
+  EXPECT_TRUE(is_contained({1,2,3,4}, 3));
+  EXPECT_FALSE(is_contained({1,2,3,4}, 5));
+
+  static_assert(is_contained({1,2,3,4}, 3), "It's there!");
+  static_assert(!is_contained({1,2,3,4}, 5), "It's not there :(");
----------------
nit: make sure to run this through clang-format if you haven't already. It looks to me like some spaces might be missing.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D122079



More information about the llvm-commits mailing list