[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 08:54:32 PDT 2022
kuhar added inline comments.
================
Comment at: llvm/include/llvm/ADT/STLExtras.h:1661
+template <typename T>
+constexpr bool is_contained(T value, std::initializer_list<T> set) {
+ // TODO: Use std::find when we switch to C++20.
----------------
This function arguments are backwards compared to the other overloads. Could we keep it consistent?
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