CodeGen : Check LLVM_ENABLE_DUMP definition for dumpMachineInstrRangeWithSlotIndex.

[flyingforyou]

Summary

Add missing check routine for dumpMachineInstrRangeWithSlotIndex including LLVM_DUMP_METHOD.

[bkramer]

This will get you an unused function warning if NDEBUG && LLVM_ENABLE_DUMP. You can use LLVM_DUMP_METHOD to avoid that.

[flyingforyou]

Addressing Benjamin's comment.
Thanks.

[bkramer]

This is fine.

[flyingforyou]

Thanks Benjamin.
Committed in r298895.

llvm/trunk/lib/CodeGen/InlineSpiller.cpp

bool InlineSpiller::coalesceStackAccess(MachineInstr *MI, unsigned Reg) {
   } else {
     ++NumSpillsRemoved;
     --NumSpills;
   }
 
   return true;
 }
 
-#if !defined(NDEBUG)
+#if !defined(NDEBUG) || defined(LLVM_ENABLE_DUMP)
+LLVM_DUMP_METHOD
 // Dump the range of instructions from B to E with their slot indexes.
 static void dumpMachineInstrRangeWithSlotIndex(MachineBasicBlock::iterator B,
                                                MachineBasicBlock::iterator E,
                                                LiveIntervals const &LIS,
                                                const char *const header,
                                                unsigned VReg =0) {
   char NextLine = '\n';
   char SlotIndent = '\t';