[PATCH] D135311: [LLVM][Support] Support for `llvm::cl::list`'s default values

Son Tuan Vu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 6 06:31:43 PDT 2022


tyb0807 added inline comments.


================
Comment at: llvm/include/llvm/Support/CommandLine.h:440
 
+template <class Ty> struct initializer_list {
+  const llvm::ArrayRef<Ty> &Inits;
----------------
zero9178 wrote:
> The naming here is a bit unfortunate since it sort of clashes with the common C++ vocabulary that is `std::initializer_list` but I don't have a lot of better ideas :( 
Maybe I'll go with `list_initializer` and `list_init` to avoid name collision


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D135311



More information about the llvm-commits mailing list