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

Reply via email to