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

Anna Zaks via cfe-commits cfe-commits at lists.llvm.org
Thu Sep 10 14:55:23 PDT 2015


zaks.anna added a comment.

Have you tested this on a large codebase that uses lambdas? When do you think we should turn this on by default?

Please, add test cases that demonstrate what happens when an issue is reported within a lambda and to check if inlined defensive checks work.

(As a follow up to this patch, we may need to teach LiveVariables.cpp and UninitializedValues.cpp about lambdas. For example, to address issues like this one: https://llvm.org/bugs/show_bug.cgi?id=22834)


http://reviews.llvm.org/D12652





More information about the cfe-commits mailing list