[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