xazax.hun added a comment.

This looks good to me, I have one question inline.

================
Comment at: lib/StaticAnalyzer/Checkers/DeadStoresChecker.cpp:425
@@ +424,3 @@
+    llvm::DenseMap<const VarDecl *, FieldDecl *> CaptureFields;
+    FieldDecl *ThisCaptureField;
+    LambdaClass->getCaptureFields(CaptureFields, ThisCaptureField);
----------------
As far as I can see ThisCaptureField is not used. Does this solution work, when 
there is a dead store to a class member (through captured this)?


http://reviews.llvm.org/D14736



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to