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

This revision was automatically updated to reflect the committed changes. Closed by commit rL365314: [analyzer] Add analyzer option to limit the number of imported TUs (authored by gamesh411, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to

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

Szelethus added inline comments. Comment at: test/Analysis/ctu-import-threshold.c:1-5 +// Ensure analyzer option 'ctu-import-threshold' is a recognized option. +// +// RUN: %clang_cc1 -analyze -analyzer-config ctu-import-threshold=30 -verify %s +// +// expected-no-diagnostics

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

martong accepted this revision. martong added inline comments. This revision is now accepted and ready to land. Comment at: include/clang/StaticAnalyzer/Core/AnalyzerOptions.def:297 +"for import when inlining functions during CTU analysis. " +

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

gamesh411 updated this revision to Diff 207712. gamesh411 added a comment. Rebase again Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D59798/new/ https://reviews.llvm.org/D59798 Files: include/clang/CrossTU/CrossTranslationUnit.h

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

gamesh411 updated this revision to Diff 207711. gamesh411 added a comment. Rebase onto current master Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D59798/new/ https://reviews.llvm.org/D59798 Files: include/clang/CrossTU/CrossTranslationUnit.h

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

gamesh411 updated this revision to Diff 199278. gamesh411 added a comment. Revert unnecessary clang-formating of AnalysisConsumer.cpp Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D59798/new/ https://reviews.llvm.org/D59798 Files:

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

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

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

gamesh411 updated this revision to Diff 199277. gamesh411 added a comment. Apply review suggestions by Xazax Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D59798/new/ https://reviews.llvm.org/D59798 Files: include/clang/CrossTU/CrossTranslationUnit.h

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

xazax.hun added a comment. Mostly looks good, I have two slightly related nits. Comment at: include/clang/CrossTU/CrossTranslationUnit.h:127 + StringRef IndexName, bool DisplayCTUProgress, + unsigned CTULoadThreshold);

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

martong added a reviewer: xazax.hun. martong added a comment. @xazax.hun Could you please take a look? Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D59798/new/ https://reviews.llvm.org/D59798 ___ cfe-commits mailing