aprantl added a comment.

In D66546#1641492 <https://reviews.llvm.org/D66546#1641492>, @clayborg wrote:

> Sounds good about LanguageSet being cheap to pass by value. Are there any 
> paths that will call this over and over where we still will be calculating 
> the LangaugeSet over and over in a type system? We might benefit from using 
> llvm::once and a static LanguageSet in the static functions that return 
> LanguageSets if that is the case?


No, the most "expensive" function is 
ClangASTContext::GetSupportedLanguagesForTypes() which is called once by the 
ClangASTContext constructor.


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

https://reviews.llvm.org/D66546



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to