[PATCH] D156016: [Support] Change MapVector's default template parameter to SmallVector<*, 0>

[Jonas Devlieghere via Phabricator via llvm-commits]

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