================ @@ -1070,57 +1077,24 @@ class Sema; }; private: - SmallVector<OverloadCandidate, 16> Candidates; - llvm::SmallPtrSet<uintptr_t, 16> Functions; - - // Allocator for ConversionSequenceLists. We store the first few of these - // inline to avoid allocation for small sets. - llvm::BumpPtrAllocator SlabAllocator; + ASTContext &Ctx; + SmallVector<OverloadCandidate, 4> Candidates; + llvm::SmallPtrSet<uintptr_t, 4> Functions; ---------------- AaronBallman wrote:
The reason it works to drop the `llvm::` prefix for `SmallVector` is because we do this: https://github.com/llvm/llvm-project/blob/main/clang/include/clang/Basic/LLVM.h `SmallPtrSet` is not included in there which is why the prefix is necessary, and I'm not certain it's worth including it or not. Thoughts? https://github.com/llvm/llvm-project/pull/88546 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits