Fix -strip-debug-declare to work when there are
authorDale Johannesen <dalej@apple.com>
Fri, 13 Mar 2009 22:59:47 +0000 (22:59 +0000)
committerDale Johannesen <dalej@apple.com>
Fri, 13 Mar 2009 22:59:47 +0000 (22:59 +0000)
llvm.global.variable's but no llvm.declare's.

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

lib/Transforms/IPO/StripSymbols.cpp

index ee6fbd3cba1064628cac0de449937a1d87bc527f..77579d7bfa083afee76e4070e3b510bc4413aa04 100644 (file)
@@ -368,29 +368,27 @@ bool StripNonDebugSymbols::runOnModule(Module &M) {
 bool StripDebugDeclare::runOnModule(Module &M) {
 
   Function *Declare = M.getFunction("llvm.dbg.declare");
-
-  if (!Declare) 
-    return false;
-
   std::vector<Constant*> DeadConstants;
 
-  while (!Declare->use_empty()) {
-    CallInst *CI = cast<CallInst>(Declare->use_back());
-    Value *Arg1 = CI->getOperand(1);
-    Value *Arg2 = CI->getOperand(2);
-    assert(CI->use_empty() && "llvm.dbg intrinsic should have void result");
-    CI->eraseFromParent();
-    if (Arg1->use_empty()) {
-      if (Constant *C = dyn_cast<Constant>(Arg1)) 
-        DeadConstants.push_back(C);
-      else 
-        RecursivelyDeleteTriviallyDeadInstructions(Arg1, NULL);
+  if (Declare) {
+    while (!Declare->use_empty()) {
+      CallInst *CI = cast<CallInst>(Declare->use_back());
+      Value *Arg1 = CI->getOperand(1);
+      Value *Arg2 = CI->getOperand(2);
+      assert(CI->use_empty() && "llvm.dbg intrinsic should have void result");
+      CI->eraseFromParent();
+      if (Arg1->use_empty()) {
+        if (Constant *C = dyn_cast<Constant>(Arg1)) 
+          DeadConstants.push_back(C);
+        else 
+          RecursivelyDeleteTriviallyDeadInstructions(Arg1, NULL);
+      }
+      if (Arg2->use_empty())
+        if (Constant *C = dyn_cast<Constant>(Arg2)) 
+          DeadConstants.push_back(C);
     }
-    if (Arg2->use_empty())
-      if (Constant *C = dyn_cast<Constant>(Arg2)) 
-        DeadConstants.push_back(C);
+    Declare->eraseFromParent();
   }
-  Declare->eraseFromParent();
 
   // Delete all llvm.dbg.global_variables.
   for (Module::global_iterator I = M.global_begin(), E = M.global_end();