[PATCH] D115064: [clang-format][NFC] Replace deque with vector
Björn Schäpers via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Dec 6 01:44:09 PST 2021
HazardyKnusperkeks added a comment.
In D115064#3172860 <https://reviews.llvm.org/D115064#3172860>, @MyDeveloperDay wrote:
> Even if you choose a number, beyond that number it behaves like a vector (just without the data being on the stack), its really good for small vectors but its not bad for large ones. I'm not sure what the default N is if you don't specify a number? (0?)
Yeah I know what this is. On the link ownpan provided it says it chooses a good number for the stack.
My initial take on it was since I don't know how many states we mostly have, I don't try to guess and waste space on the stack if there is a chance we always go to the heap anyway.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D115064/new/
https://reviews.llvm.org/D115064
More information about the cfe-commits
mailing list