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

Mehdi AMINI via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 10 14:52:30 PST 2020


mehdi_amini added a comment.

Random other proposal in the brainstorming here: introduce a subclass `llvm::Vector` that inherit from SmallVector and define the size as never exceeding 64B. `SmallVector` would always have an explicit size and `Vector` never: that at least address the case someone using `SmallVector<LargeSize>` and getting an implicit and unexpected `N=0`


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D90884



More information about the llvm-commits mailing list