MachineOperand::setIsDebug should ensure the register is /not/ a definition
authorDavid Blaikie <dblaikie@gmail.com>
Sun, 16 Jun 2013 20:34:09 +0000 (20:34 +0000)
committerDavid Blaikie <dblaikie@gmail.com>
Sun, 16 Jun 2013 20:34:09 +0000 (20:34 +0000)
This currently unused function appeared to be asserting in the wrong
direction - DebugValues are never definitions of registers, only uses.
Curiously we don't perform any of these checks for the more common (&
actually used) case of MachineOperand::CreateReg (or other Create
functions).

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

include/llvm/CodeGen/MachineOperand.h

index 414770b9ecf0f91077c965fdcf0ee1c871680dfc..57b28fecf8640253100077aec34fbc3f462547cf 100644 (file)
@@ -391,7 +391,7 @@ public:
   }
 
   void setIsDebug(bool Val = true) {
-    assert(isReg() && IsDef && "Wrong MachineOperand accessor");
+    assert(isReg() && !IsDef && "Wrong MachineOperand accessor");
     IsDebug = Val;
   }