[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