[PATCH] D156016: [Support] Change MapVector's default template parameter to SmallVector<*, 0>
Jonas Devlieghere via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jul 24 16:41:20 PDT 2023
JDevlieghere accepted this revision.
JDevlieghere added a comment.
This revision is now accepted and ready to land.
Makes sense: LGTM.
Slightly tangential: If we want to encourage more use of `SmallVector`, would it make sense to make the second template argument default to `0` so that folks can just write `SmallVector<T>` or do we want to keep it explicit so users are forced to pick a sensible value?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D156016/new/
https://reviews.llvm.org/D156016
More information about the llvm-commits
mailing list