yurai007 added a comment. In D120334#3339764 <https://reviews.llvm.org/D120334#3339764>, @serge-sans-paille wrote:
> A small local test hints that using a ``SmallVector`` instead of a > ``std::vector`` also fixes the assembly bloat. I don't know if it's worth > considering as an alternative or as an extra step. Ok, I'm gonna try SmallVectors on top of this change. LangOptions is widely used so maybe it could bring some extra gains in places where vectors are indeed filled with more data. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D120334/new/ https://reviews.llvm.org/D120334 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits