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();
+
----------------
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?
================
Comment at: lib/StaticAnalyzer/Core/MemRegion.cpp:740-741
@@ -739,3 +739,4 @@
const DeclContext *DC,
- const VarDecl *VD) {
+ const VarDecl *VD,
+ MemRegionManager *Mmgr) {
while (LC) {
----------------
jordan_rose wrote:
> Why the extra parameter?
This is just a leftover from code evolution, thank you for spotting this.
http://reviews.llvm.org/D12652
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
