[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