[PATCH] D120334: [NFC][Lexer] Use more appropriate LangOptionsBase type for Lexer::LangOpts

Wed Feb 23 06:06:13 PST 2022

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