[Lldb-commits] [PATCH] D66546: Extend FindTypes w/ CompilerContext to allow filtering by language

[Adrian Prantl via Phabricator via lldb-commits]

aprantl added a comment.

In D66546#1641435 <https://reviews.llvm.org/D66546#1641435>, @aprantl wrote:

> > Looks good overall. Just a question of it we want to return "const LanguageSet &" to avoid copies. Also switch static functions that currently return "LanguageSet" over to use static variables and llvm::once to init them and then return "const LanguageSet &".
>
> The point of the static_assert in TypeSystem.cpp is to make sure that a LanguageSet is exactly 64 bits in size so it is cheap to pass it by value.


Actually, on a 64-bit system it is *always* 64 bits in size, but the static_assert guarantees that now memory is allocated.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D66546/new/

https://reviews.llvm.org/D66546