[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