[PATCH] D115064: [clang-format][NFC] Replace deque with vector

Björn Schäpers via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Dec 5 02:10:29 PST 2021


HazardyKnusperkeks added a comment.

In D115064#3171950 <https://reviews.llvm.org/D115064#3171950>, @owenpan wrote:

> In D115064#3170615 <https://reviews.llvm.org/D115064#3170615>, @curdeius wrote:
>
>> Maybe using `llvm::SmallVector` with some well-thought (data-based) static number of elements would be better here, WDYT?
>
> I tend to agree as LLVM style <https://llvm.org/docs/ProgrammersManual.html#picking-the-right-data-structure-for-a-task> prefers `SmallVector` to `vector`.

Okay, didn't know I can use it without a number. Will update.


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

https://reviews.llvm.org/D115064



More information about the cfe-commits mailing list