From: Ted Kremenek <kremenek@apple.com>
Date: Mon, 25 Feb 2008 22:28:54 +0000 (+0000)
Subject: Minor bug fix in LiveVariables: don't "kill" decls referenced by a DeclStmt
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=dcc48100ef7c224cecf5b9ccdd5c0d39e476468a;p=clang

Minor bug fix in LiveVariables: don't "kill" decls referenced by a DeclStmt
that aren't VarDecls.


git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@47572 91177308-0d34-0410-b5e6-96231b3b80d8
---

diff --git a/Analysis/LiveVariables.cpp b/Analysis/LiveVariables.cpp
index 1201eb021d..52f7f4d19b 100644
--- a/Analysis/LiveVariables.cpp
+++ b/Analysis/LiveVariables.cpp
@@ -156,13 +156,13 @@ void TransferFuncs::VisitAssign(BinaryOperator* B) {
 void TransferFuncs::VisitDeclStmt(DeclStmt* DS) {
   // Declarations effectively "kill" a variable since they cannot
   // possibly be live before they are declared.
-  for (ScopedDecl* D = DS->getDecl(); D != NULL; D = D->getNextDeclarator()) {
-    LiveState(D,AD) = Dead;
-
-    if (VarDecl* VD = dyn_cast<VarDecl>(D))
+  for (ScopedDecl* D = DS->getDecl(); D != NULL; D = D->getNextDeclarator())
+    if (VarDecl* VD = dyn_cast<VarDecl>(D)) {
+      LiveState(D,AD) = Dead;
+      
       if (Expr* Init = VD->getInit())
         Visit(Init);
-  }
+    }
 }
   
 } // end anonymous namespace