[PATCH] [analyzer][Review request] Fix for PR18394.

Антон Ярцев anton.yartsev at gmail.com
Fri Feb 28 14:09:27 PST 2014


  Got it! Updated the patch.

Hi jordan_rose, zaks.anna, krememek,

http://llvm-reviews.chandlerc.com/D2899

CHANGE SINCE LAST DIFF
  http://llvm-reviews.chandlerc.com/D2899?vs=7435&id=7437#toc

Files:
  lib/StaticAnalyzer/Core/ExplodedGraph.cpp
  test/Analysis/NewDeleteLeaks-PR18394.cpp

Index: lib/StaticAnalyzer/Core/ExplodedGraph.cpp
===================================================================
--- lib/StaticAnalyzer/Core/ExplodedGraph.cpp
+++ lib/StaticAnalyzer/Core/ExplodedGraph.cpp
@@ -90,8 +90,9 @@
   // (7) The LocationContext is the same as the predecessor.
   // (8) Expressions that are *not* lvalue expressions.
   // (9) The PostStmt isn't for a non-consumed Stmt or Expr.
-  // (10) The successor is not a CallExpr StmtPoint (so that we would
-  //      be able to find it when retrying a call with no inlining).
+  // (10) The successor is neither a CallExpr StmtPoint nor a CallEnter or 
+  //      PreImplicitCall(so that we would be able to find it when retrying a 
+  //      call with no inlining).
   // FIXME: It may be safe to reclaim PreCall and PostCall nodes as well.
 
   // Conditions 1 and 2.
@@ -152,6 +153,10 @@
   if (Optional<StmtPoint> SP = SuccLoc.getAs<StmtPoint>())
     if (CallEvent::isCallStmt(SP->getStmt()))
       return false;
+
+  // Condition 10, continuation.
+  if (SuccLoc.getAs<CallEnter>() || progPoint.getAs<PreImplicitCall>())
+    return false;
 
   return true;
 }
Index: test/Analysis/NewDeleteLeaks-PR18394.cpp
===================================================================
--- test/Analysis/NewDeleteLeaks-PR18394.cpp
+++ test/Analysis/NewDeleteLeaks-PR18394.cpp
@@ -0,0 +1,16 @@
+// RUN: %clang_cc1 -analyzer-config graph-trim-interval=1 -analyzer-max-loop 1 -analyze -analyzer-checker=alpha.cplusplus.NewDeleteLeaks -verify %s
+// expected-no-diagnostics
+
+class A {
+public:
+  void f() {};
+  ~A() {
+    for (int i=0; i<3; i++)
+      f();
+  }
+};
+
+void error() {
+  A *a = new A();
+  delete a;
+}
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D2899.3.patch
Type: text/x-patch
Size: 1709 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140228/2e3f1a29/attachment.bin>


More information about the cfe-commits mailing list