[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