jdoerfert marked an inline comment as done. jdoerfert added inline comments.
================ Comment at: clang/include/clang/AST/OpenMPClause.h:6682 + /// The outermost level of selector sets. + llvm::SmallVector<OMPTraitSet, 4> Sets; + ---------------- rnk wrote: > This is not a good data structure choice. You have three levels of small > vector nesting, so sizeof OMPTraitInfo is 880 bytes, and then you are passing > it by value in in some of the attribute classes. Are you sure you wanted to > do that? I could make them all 0 elements long, reducing the size quite a bit for the common case of very few trait sets/selectors/properties. I could also try to dynamically allocate them once. WDYT? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D71830/new/ https://reviews.llvm.org/D71830 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
