Consider debug info intrinsics pointing to null value as dead instructions.
authorDevang Patel <dpatel@apple.com>
Fri, 18 Mar 2011 23:28:02 +0000 (23:28 +0000)
committerDevang Patel <dpatel@apple.com>
Fri, 18 Mar 2011 23:28:02 +0000 (23:28 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@127922 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Transforms/Utils/Local.cpp

index e54dfb3dc73f862a9a686d1ee7bbdde668895f8e..19b999722dbc6d94416300dc94c84f1ae3c8a318 100644 (file)
@@ -211,7 +211,20 @@ bool llvm::ConstantFoldTerminator(BasicBlock *BB) {
 bool llvm::isInstructionTriviallyDead(Instruction *I) {
   if (!I->use_empty() || isa<TerminatorInst>(I)) return false;
 
-  // We don't want debug info removed by anything this general.
+  // We don't want debug info removed by anything this general, unless
+  // debug info is empty.
+  if (DbgDeclareInst *DDI = dyn_cast<DbgDeclareInst>(I)) {
+    if (DDI->getAddress()) 
+      return false;
+    else
+      return true;
+  } else if (DbgValueInst *DVI = dyn_cast<DbgValueInst>(I)) {
+    if (DVI->getValue())
+      return false;
+    else
+      return true;
+  }
+
   if (isa<DbgInfoIntrinsic>(I)) return false;
 
   if (!I->mayHaveSideEffects()) return true;