[PATCH] D153587: [GlobPattern] Support brace expansions
Fangrui Song via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Aug 29 18:33:07 PDT 2023
MaskRay accepted this revision.
MaskRay added inline comments.
================
Comment at: llvm/include/llvm/Support/GlobPattern.h:84
+ SmallVector<Bracket, 0> Brackets;
+ std::string Pat;
+ };
----------------
ellis wrote:
> MaskRay wrote:
> > Perhaps `SmallVector<, 0>` to save memory. For the cases we care about performance, the pattern is typically longer than the max inlined element size in libc++/libstdc++ (15 or 23).
> I'm curious, why not leave out the `0` and just have `SmallVector<char> Pat`?
I consider it unfortunate... Omitting the argument has been reserved to select an inline size to have about 64 bytes IIUC.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D153587/new/
https://reviews.llvm.org/D153587
More information about the llvm-commits
mailing list