[PATCH] D59798: [analyzer] Add analyzer option to limit the number of imported TUs

Endre Fülöp via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon May 13 09:24:26 PDT 2019


gamesh411 marked an inline comment as done.
gamesh411 added a comment.

I could greatly simplify the API of getCrossTUDefinition by injecting the threshold value in the constructor. A minor drawback with this approach is that testing code is a bit more complicated, as I had to patch the CompilerInstance to override the AnalyzerOption value for CTUImportThreshold. Thanks for the suggestion!


Repository:
  rC Clang

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

https://reviews.llvm.org/D59798





More information about the cfe-commits mailing list