[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