Author: Endre Fülöp Date: 2020-07-01T10:08:52+02:00 New Revision: 52f65323660051a5d039d475edfd4a3018682dcb
URL: https://github.com/llvm/llvm-project/commit/52f65323660051a5d039d475edfd4a3018682dcb DIFF: https://github.com/llvm/llvm-project/commit/52f65323660051a5d039d475edfd4a3018682dcb.diff LOG: [analyzer][CrossTU] Lower CTUImportThreshold default value Summary: The default value of 100 makes the analysis slow. Projects of considerable size can take more time to finish than it is practical. The new default setting of 8 is based on the analysis of LLVM itself. With the old default value of 100 the analysis time was over a magnitude slower. Thresholding the load of ASTUnits is to be extended in the future with a more fine-tuneable solution that accomodates to the specifics of the project analyzed. Reviewers: martong, balazske, Szelethus Subscribers: whisperity, xazax.hun, baloghadamsoftware, szepet, rnkovacs, a.sidorin, mikhail.ramalho, Szelethus, donat.nagy, dkrupp, Charusso, steakhal, ASDenysPetrov, cfe-commits Tags: #clang Differential Revision: https://reviews.llvm.org/D82561 Added: Modified: clang/include/clang/StaticAnalyzer/Core/AnalyzerOptions.def clang/test/Analysis/analyzer-config.c Removed: ################################################################################ diff --git a/clang/include/clang/StaticAnalyzer/Core/AnalyzerOptions.def b/clang/include/clang/StaticAnalyzer/Core/AnalyzerOptions.def index 8944dfe0f749..9ee113c0dcaf 100644 --- a/clang/include/clang/StaticAnalyzer/Core/AnalyzerOptions.def +++ b/clang/include/clang/StaticAnalyzer/Core/AnalyzerOptions.def @@ -324,7 +324,7 @@ ANALYZER_OPTION(unsigned, CTUImportThreshold, "ctu-import-threshold", "Lowering this threshold can alleviate the memory burder of " "analysis with many interdependent definitions located in " "various translation units.", - 100u) + 8u) ANALYZER_OPTION( unsigned, AlwaysInlineSize, "ipa-always-inline-size", diff --git a/clang/test/Analysis/analyzer-config.c b/clang/test/Analysis/analyzer-config.c index 7a411a162201..e4035cf755b2 100644 --- a/clang/test/Analysis/analyzer-config.c +++ b/clang/test/Analysis/analyzer-config.c @@ -41,7 +41,7 @@ // CHECK-NEXT: cplusplus.Move:WarnOn = KnownsAndLocals // CHECK-NEXT: crosscheck-with-z3 = false // CHECK-NEXT: ctu-dir = "" -// CHECK-NEXT: ctu-import-threshold = 100 +// CHECK-NEXT: ctu-import-threshold = 8 // CHECK-NEXT: ctu-index-name = externalDefMap.txt // CHECK-NEXT: ctu-invocation-list = invocations.yaml // CHECK-NEXT: deadcode.DeadStores:ShowFixIts = false _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits