[PATCH] D92522: [SmallVector] Allow SmallVector<T>

Sean Silva via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 3 10:42:11 PST 2020


silvas added inline comments.


================
Comment at: llvm/docs/ProgrammersManual.rst:1527
+omitting the ``N``). This will automatically choose a reasonable number of
+inlined elements.
+
----------------
lattner wrote:
> I expand on what reasonable means here.  I'd say "reasonable for inline allocation on the stack" or something like that, maybe even give the target number of inline bytes so people have a mental model.
Good point. I incorporated your suggestions.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D92522



More information about the llvm-commits mailing list