[PATCH] D90884: [SmallVector] Add a default small size.

[Sean Silva via Phabricator via llvm-commits]

silvas added a comment.

Also, agreed that DefaultSmallVector is quite a mouthful and unlikely to catch on.

Maybe `llvm::SVector` / `llvm::SmallVec`? That's more succinct (helps it to catch on, and echoes the intended "default" / "convenience" aspect) but still carries a name that lets people know that they are getting inline storage. Precedent for `SVector` is `raw_svector_ostream`. Personally `SmallVec` reads a bit nicer to me.

  template <class T>
  using SVector/SmallVec = SmallVector<T, kDefaultSmallSize<T>>;


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D90884