[llvm-commits] [llvm] r44756 - /llvm/trunk/lib/VMCore/Value.cpp

[Chris Lattner]

Author: lattner
Date: Sun Dec  9 19:51:22 2007
New Revision: 44756

URL: http://llvm.org/viewvc/llvm-project?rev=44756&view=rev
Log:
fix some warnings.

Modified:
    llvm/trunk/lib/VMCore/Value.cpp

Modified: llvm/trunk/lib/VMCore/Value.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/VMCore/Value.cpp?rev=44756&r1=44755&r2=44756&view=diff

==============================================================================
--- llvm/trunk/lib/VMCore/Value.cpp (original)
+++ llvm/trunk/lib/VMCore/Value.cpp Sun Dec  9 19:51:22 2007
@@ -140,9 +140,9 @@
       CallInst::destroyThis(CI);
     else if (CmpInst *CI = dyn_cast<CmpInst>(this))
     {
-      if (FCmpInst *FCI = dyn_cast<FCmpInst>(this))
+      if (FCmpInst *FCI = dyn_cast<FCmpInst>(CI))
         FCmpInst::destroyThis(FCI);
-      else if (ICmpInst *ICI = dyn_cast<ICmpInst>(this))
+      else if (ICmpInst *ICI = dyn_cast<ICmpInst>(CI))
         ICmpInst::destroyThis(ICI);
       else
         assert(0 && "Unknown CmpInst-inherited class in ~Value.");
@@ -163,28 +163,25 @@
       StoreInst::destroyThis(SI);
     else if (TerminatorInst *TI = dyn_cast<TerminatorInst>(this))
     {
-      if (BranchInst* BI = dyn_cast<BranchInst>(this))
+      if (BranchInst* BI = dyn_cast<BranchInst>(TI))
         BranchInst::destroyThis(BI);
-      else if (InvokeInst* II = dyn_cast<InvokeInst>(this))
+      else if (InvokeInst* II = dyn_cast<InvokeInst>(TI))
         InvokeInst::destroyThis(II);
-      else if (ReturnInst* RI = dyn_cast<ReturnInst>(this))
+      else if (ReturnInst* RI = dyn_cast<ReturnInst>(TI))
         ReturnInst::destroyThis(RI);
-      else if (SwitchInst *SI = dyn_cast<SwitchInst>(this))
+      else if (SwitchInst *SI = dyn_cast<SwitchInst>(TI))
         SwitchInst::destroyThis(SI);
-      else if (UnreachableInst *UI = dyn_cast<UnreachableInst>(this))
+      else if (UnreachableInst *UI = dyn_cast<UnreachableInst>(TI))
         UnreachableInst::destroyThis(UI);
-      else if (UnwindInst *UI = dyn_cast<UnwindInst>(this))
+      else if (UnwindInst *UI = dyn_cast<UnwindInst>(TI))
         UnwindInst::destroyThis(UI);
       else
         assert(0 && "Unknown TerminatorInst-inherited class in ~Value.");
-    }
-    else if(UnaryInstruction* UI = dyn_cast<UnaryInstruction>(this))
-    {
-      if(AllocationInst* AI = dyn_cast<AllocationInst>(this))
-      {
-        if(AllocaInst* AI = dyn_cast<AllocaInst>(this))
+    } else if(UnaryInstruction* UI = dyn_cast<UnaryInstruction>(this)) {
+      if(AllocationInst* AI = dyn_cast<AllocationInst>(UI)) {
+        if(AllocaInst* AI = dyn_cast<AllocaInst>(UI))
           AllocaInst::destroyThis(AI);
-        else if(MallocInst* MI = dyn_cast<MallocInst>(this))
+        else if(MallocInst* MI = dyn_cast<MallocInst>(UI))
           MallocInst::destroyThis(MI);
         else
           assert(0 && "Unknown AllocationInst-inherited class in ~Value.");