diff options
| author | Matthias Braun <matze@braunis.de> | 2017-01-28 02:02:38 +0000 |
|---|---|---|
| committer | Matthias Braun <matze@braunis.de> | 2017-01-28 02:02:38 +0000 |
| commit | 8c209aa8779de57771b64896f805e14cc0016dcd (patch) | |
| tree | b3f7bc33dbb7be8ffe9a4f974253d6547b94ebdb /llvm/lib/Target/MSP430/MSP430MCInstLower.cpp | |
| parent | cfa5dc4a1dc2b7495aba96caba6fd2600c1ceb8a (diff) | |
| download | bcm5719-llvm-8c209aa8779de57771b64896f805e14cc0016dcd.tar.gz bcm5719-llvm-8c209aa8779de57771b64896f805e14cc0016dcd.zip | |
Cleanup dump() functions.
We had various variants of defining dump() functions in LLVM. Normalize
them (this should just consistently implement the things discussed in
http://lists.llvm.org/pipermail/cfe-dev/2014-January/034323.html
For reference:
- Public headers should just declare the dump() method but not use
LLVM_DUMP_METHOD or #if !defined(NDEBUG) || defined(LLVM_ENABLE_DUMP)
- The definition of a dump method should look like this:
#if !defined(NDEBUG) || defined(LLVM_ENABLE_DUMP)
LLVM_DUMP_METHOD void MyClass::dump() {
// print stuff to dbgs()...
}
#endif
llvm-svn: 293359
Diffstat (limited to 'llvm/lib/Target/MSP430/MSP430MCInstLower.cpp')
| -rw-r--r-- | llvm/lib/Target/MSP430/MSP430MCInstLower.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm/lib/Target/MSP430/MSP430MCInstLower.cpp b/llvm/lib/Target/MSP430/MSP430MCInstLower.cpp index 47b0e270c5b..e7716382b22 100644 --- a/llvm/lib/Target/MSP430/MSP430MCInstLower.cpp +++ b/llvm/lib/Target/MSP430/MSP430MCInstLower.cpp @@ -119,7 +119,7 @@ void MSP430MCInstLower::Lower(const MachineInstr *MI, MCInst &OutMI) const { MCOperand MCOp; switch (MO.getType()) { default: - MI->dump(); + MI->print(errs()); llvm_unreachable("unknown operand type"); case MachineOperand::MO_Register: // Ignore all implicit register operands. |

