[PATCH] D12652: [Static Analyzer] Lambda support.

Gábor Horváth via cfe-commits cfe-commits at lists.llvm.org
Fri Sep 11 09:58:15 PDT 2015


xazax.hun added inline comments.

================
Comment at: include/clang/StaticAnalyzer/Core/AnalyzerOptions.h:515-517
@@ -511,1 +514,5 @@
 
+  /// Returns true if lambdas should be inlined. Otherwise a sink node will be
+  /// generated each time a LambdaExpr is visited.
+  bool shouldInlineLambdas();
+
----------------
xazax.hun wrote:
> jordan_rose wrote:
> > "inline" is kind of a misnomer, since we may not actually inline lambdas. I would have suggested "model lambdas" or "lambda support".
> Even when this configuration option is set to false, the body of the lambda is analyzed as a top level function. For this reason I think the "lambda support" might be a misnomer too. What do you think?
If we come up with a better name for the option I will address that in a separate patch.


Repository:
  rL LLVM

http://reviews.llvm.org/D12652





More information about the cfe-commits mailing list