================
@@ -808,7 +808,7 @@ class AttributePool {
 
 class ParsedAttributesView {
   friend class AttributePool;
-  using VecTy = llvm::SmallVector<ParsedAttr *>;
+  using VecTy = llvm::SmallVector<ParsedAttr *, 2>;
----------------
erichkeane wrote:

Curious why '2' was found?  Did you see perf-benefit from 2 over 0(aka, 
`std::vector`)?  Presumably the benefit of cache locality isn't really 
applicable as attributes are comparatively rare, so 'while we're at it', 
perhaps saving twice as much would be valuable? 

https://github.com/llvm/llvm-project/pull/148726
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to