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/BPF/BPFMCInstLower.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/BPF/BPFMCInstLower.cpp')
-rw-r--r-- | llvm/lib/Target/BPF/BPFMCInstLower.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm/lib/Target/BPF/BPFMCInstLower.cpp b/llvm/lib/Target/BPF/BPFMCInstLower.cpp index 1faa1c09765..c8528e86731 100644 --- a/llvm/lib/Target/BPF/BPFMCInstLower.cpp +++ b/llvm/lib/Target/BPF/BPFMCInstLower.cpp @@ -54,7 +54,7 @@ void BPFMCInstLower::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. |