[PATCH] D89544: [CodingStandards] Clarify the recommendation to use SmallVector

John McCall via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 16 09:55:51 PDT 2020


rjmccall added a comment.

`llvm::SmallVector` isn't, like, otherwise worse than `std::vector`, so that you should only consider it over `std::vector` when the inline-capacity optimization is highly valuable.  It does have some specific drawbacks, but we go into this in more detail in the programmer's manual.  I think "prefer llvm::SmallVector" is fine general advice for this paragraph.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89544



More information about the llvm-commits mailing list