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



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

Reply via email to