[cfe-commits] r152072 - in /cfe/trunk/include/clang: Driver/CC1Options.td Frontend/AnalyzerOptions.h

Anna Zaks ganna at apple.com
Mon Mar 5 15:54:56 PST 2012


Author: zaks
Date: Mon Mar  5 17:54:55 2012
New Revision: 152072

URL: http://llvm.org/viewvc/llvm-project?rev=152072&view=rev
Log:
[analyzer]  Bump up the size of the functions that should be
considered for inlining to 200 BBs.

Setting the max to 10 BBs introduced several false negatives, we'll
reevaluate the setting later on along with other inlining heuristics.

Modified:
    cfe/trunk/include/clang/Driver/CC1Options.td
    cfe/trunk/include/clang/Frontend/AnalyzerOptions.h

Modified: cfe/trunk/include/clang/Driver/CC1Options.td
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Driver/CC1Options.td?rev=152072&r1=152071&r2=152072&view=diff
==============================================================================
--- cfe/trunk/include/clang/Driver/CC1Options.td (original)
+++ cfe/trunk/include/clang/Driver/CC1Options.td Mon Mar  5 17:54:55 2012
@@ -85,7 +85,7 @@
 def analyzer_inline_max_stack_depth : Separate<"-analyzer-inline-max-stack-depth">,
   HelpText<"Bound on stack depth while inlining (4 by default)">;
 def analyzer_inline_max_function_size : Separate<"-analyzer-inline-max-function-size">,
-  HelpText<"Bound on the number of basic blocks in an inlined function (10 by default)">;
+  HelpText<"Bound on the number of basic blocks in an inlined function (200 by default)">;
 def analyzer_max_nodes : Separate<"-analyzer-max-nodes">,
   HelpText<"The maximum number of nodes the analyzer can generate (150000 default, 0 = no limit)">;
 def analyzer_max_loop : Separate<"-analyzer-max-loop">,

Modified: cfe/trunk/include/clang/Frontend/AnalyzerOptions.h
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Frontend/AnalyzerOptions.h?rev=152072&r1=152071&r2=152072&view=diff
==============================================================================
--- cfe/trunk/include/clang/Frontend/AnalyzerOptions.h (original)
+++ cfe/trunk/include/clang/Frontend/AnalyzerOptions.h Mon Mar  5 17:54:55 2012
@@ -110,7 +110,7 @@
     PrintStats = 0;
     // Cap the stack depth at 4 calls (5 stack frames, base + 4 calls).
     InlineMaxStackDepth = 5;
-    InlineMaxFunctionSize = 10;
+    InlineMaxFunctionSize = 200;
   }
 };
 





More information about the cfe-commits mailing list