Wouldn't dead code stripping in the linker have removed these from a released build?<br><br><div class="gmail_quote">On Fri, Sep 7, 2012 at 4:07 AM, Duncan Sands <span dir="ltr"><<a href="mailto:baldrick@free.fr" target="_blank">baldrick@free.fr</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Manman,<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Release build: guard dump functions with "ifndef NDEBUG"<br>
</blockquote>
<br>
is this a good idea?  It is sometimes necessary to debug release builds, in<br>
which case it is helpful to be able to call "dump" routines from the debugger.<br>
<br>
Ciao, Duncan.<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
No functional change.<br>
<br>
Modified:<br>
     llvm/trunk/lib/CodeGen/<u></u>LiveInterval.cpp<br>
     llvm/trunk/lib/CodeGen/<u></u>LiveIntervalAnalysis.cpp<br>
     llvm/trunk/lib/CodeGen/<u></u>LiveVariables.cpp<br>
     llvm/trunk/lib/CodeGen/<u></u>MachineBasicBlock.cpp<br>
     llvm/trunk/lib/CodeGen/<u></u>MachineFunction.cpp<br>
     llvm/trunk/lib/CodeGen/<u></u>MachineInstr.cpp<br>
     llvm/trunk/lib/CodeGen/<u></u>MachineLoopInfo.cpp<br>
     llvm/trunk/lib/CodeGen/<u></u>MachineScheduler.cpp<br>
     llvm/trunk/lib/CodeGen/<u></u>PostRASchedulerList.cpp<br>
     llvm/trunk/lib/CodeGen/<u></u>RegisterPressure.cpp<br>
     llvm/trunk/lib/CodeGen/<u></u>ScheduleDAG.cpp<br>
     llvm/trunk/lib/CodeGen/<u></u>ScheduleDAGInstrs.cpp<br>
     llvm/trunk/lib/CodeGen/<u></u>ScoreboardHazardRecognizer.cpp<br>
     llvm/trunk/lib/CodeGen/<u></u>SelectionDAG/<u></u>ScheduleDAGRRList.cpp<br>
     llvm/trunk/lib/CodeGen/<u></u>SelectionDAG/<u></u>ScheduleDAGSDNodes.cpp<br>
     llvm/trunk/lib/CodeGen/<u></u>SlotIndexes.cpp<br>
     llvm/trunk/lib/CodeGen/<u></u>SplitKit.cpp<br>
     llvm/trunk/lib/CodeGen/<u></u>VirtRegMap.cpp<br>
     llvm/trunk/lib/Target/X86/<u></u>X86FloatingPoint.cpp<br>
     llvm/trunk/lib/Target/X86/<u></u>X86ISelDAGToDAG.cpp<br>
<br>
Modified: llvm/trunk/lib/CodeGen/<u></u>LiveInterval.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/LiveInterval.cpp?rev=163339&r1=163338&r2=163339&view=diff" target="_blank">http://llvm.org/viewvc/llvm-<u></u>project/llvm/trunk/lib/<u></u>CodeGen/LiveInterval.cpp?rev=<u></u>163339&r1=163338&r2=163339&<u></u>view=diff</a><br>

==============================<u></u>==============================<u></u>==================<br>
--- llvm/trunk/lib/CodeGen/<u></u>LiveInterval.cpp (original)<br>
+++ llvm/trunk/lib/CodeGen/<u></u>LiveInterval.cpp Thu Sep  6 14:06:06 2012<br>
@@ -748,9 +748,11 @@<br>
    return os << '[' << LR.start << ',' << LR.end << ':' << LR.valno->id << ")";<br>
  }<br>
<br>
+#ifndef NDEBUG<br>
  void LiveRange::dump() const {<br>
    dbgs() << *this << "\n";<br>
  }<br>
+#endif<br>
<br>
  void LiveInterval::print(raw_<u></u>ostream &OS) const {<br>
    if (empty())<br>
@@ -783,9 +785,11 @@<br>
    }<br>
  }<br>
<br>
+#ifndef NDEBUG<br>
  void LiveInterval::dump() const {<br>
    dbgs() << *this << "\n";<br>
  }<br>
+#endif<br>
<br>
  #ifndef NDEBUG<br>
  void LiveInterval::verify() const {<br>
<br>
Modified: llvm/trunk/lib/CodeGen/<u></u>LiveIntervalAnalysis.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/LiveIntervalAnalysis.cpp?rev=163339&r1=163338&r2=163339&view=diff" target="_blank">http://llvm.org/viewvc/llvm-<u></u>project/llvm/trunk/lib/<u></u>CodeGen/LiveIntervalAnalysis.<u></u>cpp?rev=163339&r1=163338&r2=<u></u>163339&view=diff</a><br>

==============================<u></u>==============================<u></u>==================<br>
--- llvm/trunk/lib/CodeGen/<u></u>LiveIntervalAnalysis.cpp (original)<br>
+++ llvm/trunk/lib/CodeGen/<u></u>LiveIntervalAnalysis.cpp Thu Sep  6 14:06:06 2012<br>
@@ -156,9 +156,11 @@<br>
    MF->print(OS, Indexes);<br>
  }<br>
<br>
+#ifndef NDEBUG<br>
  void LiveIntervals::dumpInstrs() const {<br>
    printInstrs(dbgs());<br>
  }<br>
+#endif<br>
<br>
  static<br>
  bool MultipleDefsBySameMI(const MachineInstr &MI, unsigned MOIdx) {<br>
<br>
Modified: llvm/trunk/lib/CodeGen/<u></u>LiveVariables.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/LiveVariables.cpp?rev=163339&r1=163338&r2=163339&view=diff" target="_blank">http://llvm.org/viewvc/llvm-<u></u>project/llvm/trunk/lib/<u></u>CodeGen/LiveVariables.cpp?rev=<u></u>163339&r1=163338&r2=163339&<u></u>view=diff</a><br>

==============================<u></u>==============================<u></u>==================<br>
--- llvm/trunk/lib/CodeGen/<u></u>LiveVariables.cpp (original)<br>
+++ llvm/trunk/lib/CodeGen/<u></u>LiveVariables.cpp Thu Sep  6 14:06:06 2012<br>
@@ -65,6 +65,7 @@<br>
  }<br>
<br>
  void LiveVariables::VarInfo::dump() const {<br>
+#ifndef NDEBUG<br>
    dbgs() << "  Alive in blocks: ";<br>
    for (SparseBitVector<>::iterator I = AliveBlocks.begin(),<br>
             E = AliveBlocks.end(); I != E; ++I)<br>
@@ -77,6 +78,7 @@<br>
        dbgs() << "\n    #" << i << ": " << *Kills[i];<br>
      dbgs() << "\n";<br>
    }<br>
+#endif<br>
  }<br>
<br>
  /// getVarInfo - Get (possibly creating) a VarInfo object for the given vreg.<br>
<br>
Modified: llvm/trunk/lib/CodeGen/<u></u>MachineBasicBlock.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/MachineBasicBlock.cpp?rev=163339&r1=163338&r2=163339&view=diff" target="_blank">http://llvm.org/viewvc/llvm-<u></u>project/llvm/trunk/lib/<u></u>CodeGen/MachineBasicBlock.cpp?<u></u>rev=163339&r1=163338&r2=<u></u>163339&view=diff</a><br>

==============================<u></u>==============================<u></u>==================<br>
--- llvm/trunk/lib/CodeGen/<u></u>MachineBasicBlock.cpp (original)<br>
+++ llvm/trunk/lib/CodeGen/<u></u>MachineBasicBlock.cpp Thu Sep  6 14:06:06 2012<br>
@@ -228,9 +228,11 @@<br>
    return 0;<br>
  }<br>
<br>
+#ifndef NDEBUG<br>
  void MachineBasicBlock::dump() const {<br>
    print(dbgs());<br>
  }<br>
+#endif<br>
<br>
  StringRef MachineBasicBlock::getName() const {<br>
    if (const BasicBlock *LBB = getBasicBlock())<br>
<br>
Modified: llvm/trunk/lib/CodeGen/<u></u>MachineFunction.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/MachineFunction.cpp?rev=163339&r1=163338&r2=163339&view=diff" target="_blank">http://llvm.org/viewvc/llvm-<u></u>project/llvm/trunk/lib/<u></u>CodeGen/MachineFunction.cpp?<u></u>rev=163339&r1=163338&r2=<u></u>163339&view=diff</a><br>

==============================<u></u>==============================<u></u>==================<br>
--- llvm/trunk/lib/CodeGen/<u></u>MachineFunction.cpp (original)<br>
+++ llvm/trunk/lib/CodeGen/<u></u>MachineFunction.cpp Thu Sep  6 14:06:06 2012<br>
@@ -284,9 +284,11 @@<br>
    return std::make_pair(Result, Result + Num);<br>
  }<br>
<br>
+#ifndef NDEBUG<br>
  void MachineFunction::dump() const {<br>
    print(dbgs());<br>
  }<br>
+#endif<br>
<br>
  StringRef MachineFunction::getName() const {<br>
    assert(getFunction() && "No function!");<br>
@@ -532,9 +534,11 @@<br>
    }<br>
  }<br>
<br>
+#ifndef NDEBUG<br>
  void MachineFrameInfo::dump(const MachineFunction &MF) const {<br>
    print(MF, dbgs());<br>
  }<br>
+#endif<br>
<br>
  //===-------------------------<u></u>------------------------------<u></u>---------------===//<br>
  //  MachineJumpTableInfo implementation<br>
@@ -629,7 +633,9 @@<br>
    OS << '\n';<br>
  }<br>
<br>
+#ifndef NDEBUG<br>
  void MachineJumpTableInfo::dump() const { print(dbgs()); }<br>
+#endif<br>
<br>
<br>
  //===-------------------------<u></u>------------------------------<u></u>---------------===//<br>
@@ -762,4 +768,6 @@<br>
    }<br>
  }<br>
<br>
+#ifndef NDEBUG<br>
  void MachineConstantPool::dump() const { print(dbgs()); }<br>
+#endif<br>
<br>
Modified: llvm/trunk/lib/CodeGen/<u></u>MachineInstr.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/MachineInstr.cpp?rev=163339&r1=163338&r2=163339&view=diff" target="_blank">http://llvm.org/viewvc/llvm-<u></u>project/llvm/trunk/lib/<u></u>CodeGen/MachineInstr.cpp?rev=<u></u>163339&r1=163338&r2=163339&<u></u>view=diff</a><br>

==============================<u></u>==============================<u></u>==================<br>
--- llvm/trunk/lib/CodeGen/<u></u>MachineInstr.cpp (original)<br>
+++ llvm/trunk/lib/CodeGen/<u></u>MachineInstr.cpp Thu Sep  6 14:06:06 2012<br>
@@ -1496,7 +1496,9 @@<br>
  }<br>
<br>
  void MachineInstr::dump() const {<br>
+#ifndef NDEBUG<br>
    dbgs() << "  " << *this;<br>
+#endif<br>
  }<br>
<br>
  static void printDebugLoc(DebugLoc DL, const MachineFunction *MF,<br>
<br>
Modified: llvm/trunk/lib/CodeGen/<u></u>MachineLoopInfo.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/MachineLoopInfo.cpp?rev=163339&r1=163338&r2=163339&view=diff" target="_blank">http://llvm.org/viewvc/llvm-<u></u>project/llvm/trunk/lib/<u></u>CodeGen/MachineLoopInfo.cpp?<u></u>rev=163339&r1=163338&r2=<u></u>163339&view=diff</a><br>

==============================<u></u>==============================<u></u>==================<br>
--- llvm/trunk/lib/CodeGen/<u></u>MachineLoopInfo.cpp (original)<br>
+++ llvm/trunk/lib/CodeGen/<u></u>MachineLoopInfo.cpp Thu Sep  6 14:06:06 2012<br>
@@ -74,6 +74,8 @@<br>
    return BotMBB;<br>
  }<br>
<br>
+#ifndef NDEBUG<br>
  void MachineLoop::dump() const {<br>
    print(dbgs());<br>
  }<br>
+#endif<br>
<br>
Modified: llvm/trunk/lib/CodeGen/<u></u>MachineScheduler.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/MachineScheduler.cpp?rev=163339&r1=163338&r2=163339&view=diff" target="_blank">http://llvm.org/viewvc/llvm-<u></u>project/llvm/trunk/lib/<u></u>CodeGen/MachineScheduler.cpp?<u></u>rev=163339&r1=163338&r2=<u></u>163339&view=diff</a><br>

==============================<u></u>==============================<u></u>==================<br>
--- llvm/trunk/lib/CodeGen/<u></u>MachineScheduler.cpp (original)<br>
+++ llvm/trunk/lib/CodeGen/<u></u>MachineScheduler.cpp Thu Sep  6 14:06:06 2012<br>
@@ -764,12 +764,14 @@<br>
      Queue.pop_back();<br>
    }<br>
<br>
+#ifndef NDEBUG<br>
    void dump() {<br>
      dbgs() << Name << ": ";<br>
      for (unsigned i = 0, e = Queue.size(); i < e; ++i)<br>
        dbgs() << Queue[i]->NodeNum << " ";<br>
      dbgs() << "\n";<br>
    }<br>
+#endif<br>
  };<br>
<br>
  /// ConvergingScheduler shrinks the unscheduled zone using heuristics to balance<br>
<br>
Modified: llvm/trunk/lib/CodeGen/<u></u>PostRASchedulerList.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/PostRASchedulerList.cpp?rev=163339&r1=163338&r2=163339&view=diff" target="_blank">http://llvm.org/viewvc/llvm-<u></u>project/llvm/trunk/lib/<u></u>CodeGen/PostRASchedulerList.<u></u>cpp?rev=163339&r1=163338&r2=<u></u>163339&view=diff</a><br>

==============================<u></u>==============================<u></u>==================<br>
--- llvm/trunk/lib/CodeGen/<u></u>PostRASchedulerList.cpp (original)<br>
+++ llvm/trunk/lib/CodeGen/<u></u>PostRASchedulerList.cpp Thu Sep  6 14:06:06 2012<br>
@@ -240,6 +240,7 @@<br>
    ScheduleDAGInstrs::exitRegion(<u></u>);<br>
  }<br>
<br>
+#ifndef NDEBUG<br>
  /// dumpSchedule - dump the scheduled Sequence.<br>
  void SchedulePostRATDList::<u></u>dumpSchedule() const {<br>
    for (unsigned i = 0, e = Sequence.size(); i != e; i++) {<br>
@@ -249,6 +250,7 @@<br>
        dbgs() << "**** NOOP ****\n";<br>
    }<br>
  }<br>
+#endif<br>
<br>
  bool PostRAScheduler::<u></u>runOnMachineFunction(<u></u>MachineFunction &Fn) {<br>
    TII = Fn.getTarget().getInstrInfo();<br>
<br>
Modified: llvm/trunk/lib/CodeGen/<u></u>RegisterPressure.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/RegisterPressure.cpp?rev=163339&r1=163338&r2=163339&view=diff" target="_blank">http://llvm.org/viewvc/llvm-<u></u>project/llvm/trunk/lib/<u></u>CodeGen/RegisterPressure.cpp?<u></u>rev=163339&r1=163338&r2=<u></u>163339&view=diff</a><br>

==============================<u></u>==============================<u></u>==================<br>
--- llvm/trunk/lib/CodeGen/<u></u>RegisterPressure.cpp (original)<br>
+++ llvm/trunk/lib/CodeGen/<u></u>RegisterPressure.cpp Thu Sep  6 14:06:06 2012<br>
@@ -63,6 +63,7 @@<br>
    decreaseSetPressure(<u></u>MaxSetPressure, RC, TRI);<br>
  }<br>
<br>
+#ifndef NDEBUG<br>
  void RegisterPressure::dump(const TargetRegisterInfo *TRI) {<br>
    dbgs() << "Live In: ";<br>
    for (unsigned i = 0, e = LiveInRegs.size(); i < e; ++i)<br>
@@ -78,6 +79,7 @@<br>
               << '\n';<br>
    }<br>
  }<br>
+#endif<br>
<br>
  /// Increase the current pressure as impacted by these physical registers and<br>
  /// bump the high water mark if needed.<br>
<br>
Modified: llvm/trunk/lib/CodeGen/<u></u>ScheduleDAG.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/ScheduleDAG.cpp?rev=163339&r1=163338&r2=163339&view=diff" target="_blank">http://llvm.org/viewvc/llvm-<u></u>project/llvm/trunk/lib/<u></u>CodeGen/ScheduleDAG.cpp?rev=<u></u>163339&r1=163338&r2=163339&<u></u>view=diff</a><br>

==============================<u></u>==============================<u></u>==================<br>
--- llvm/trunk/lib/CodeGen/<u></u>ScheduleDAG.cpp (original)<br>
+++ llvm/trunk/lib/CodeGen/<u></u>ScheduleDAG.cpp Thu Sep  6 14:06:06 2012<br>
@@ -279,6 +279,7 @@<br>
    } while (!WorkList.empty());<br>
  }<br>
<br>
+#ifndef NDEBUG<br>
  /// SUnit - Scheduling unit. It's an wrapper around either a single SDNode or<br>
  /// a group of nodes flagged together.<br>
  void SUnit::dump(const ScheduleDAG *G) const {<br>
@@ -336,6 +337,7 @@<br>
    }<br>
    dbgs() << "\n";<br>
  }<br>
+#endif<br>
<br>
  #ifndef NDEBUG<br>
  /// VerifyScheduledDAG - Verify that all SUnits were scheduled and that<br>
<br>
Modified: llvm/trunk/lib/CodeGen/<u></u>ScheduleDAGInstrs.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/ScheduleDAGInstrs.cpp?rev=163339&r1=163338&r2=163339&view=diff" target="_blank">http://llvm.org/viewvc/llvm-<u></u>project/llvm/trunk/lib/<u></u>CodeGen/ScheduleDAGInstrs.cpp?<u></u>rev=163339&r1=163338&r2=<u></u>163339&view=diff</a><br>

==============================<u></u>==============================<u></u>==================<br>
--- llvm/trunk/lib/CodeGen/<u></u>ScheduleDAGInstrs.cpp (original)<br>
+++ llvm/trunk/lib/CodeGen/<u></u>ScheduleDAGInstrs.cpp Thu Sep  6 14:06:06 2012<br>
@@ -1012,7 +1012,9 @@<br>
  }<br>
<br>
  void ScheduleDAGInstrs::dumpNode(<u></u>const SUnit *SU) const {<br>
+#ifndef NDEBUG<br>
    SU->getInstr()->dump();<br>
+#endif<br>
  }<br>
<br>
  std::string ScheduleDAGInstrs::<u></u>getGraphNodeLabel(const SUnit *SU) const {<br>
<br>
Modified: llvm/trunk/lib/CodeGen/<u></u>ScoreboardHazardRecognizer.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/ScoreboardHazardRecognizer.cpp?rev=163339&r1=163338&r2=163339&view=diff" target="_blank">http://llvm.org/viewvc/llvm-<u></u>project/llvm/trunk/lib/<u></u>CodeGen/<u></u>ScoreboardHazardRecognizer.<u></u>cpp?rev=163339&r1=163338&r2=<u></u>163339&view=diff</a><br>

==============================<u></u>==============================<u></u>==================<br>
--- llvm/trunk/lib/CodeGen/<u></u>ScoreboardHazardRecognizer.cpp (original)<br>
+++ llvm/trunk/lib/CodeGen/<u></u>ScoreboardHazardRecognizer.cpp Thu Sep  6 14:06:06 2012<br>
@@ -89,6 +89,7 @@<br>
    ReservedScoreboard.reset();<br>
  }<br>
<br>
+#ifndef NDEBUG<br>
  void ScoreboardHazardRecognizer::<u></u>Scoreboard::dump() const {<br>
    dbgs() << "Scoreboard:\n";<br>
<br>
@@ -104,6 +105,7 @@<br>
      dbgs() << '\n';<br>
    }<br>
  }<br>
+#endif<br>
<br>
  bool ScoreboardHazardRecognizer::<u></u>atIssueLimit() const {<br>
    if (IssueWidth == 0)<br>
<br>
Modified: llvm/trunk/lib/CodeGen/<u></u>SelectionDAG/<u></u>ScheduleDAGRRList.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/SelectionDAG/ScheduleDAGRRList.cpp?rev=163339&r1=163338&r2=163339&view=diff" target="_blank">http://llvm.org/viewvc/llvm-<u></u>project/llvm/trunk/lib/<u></u>CodeGen/SelectionDAG/<u></u>ScheduleDAGRRList.cpp?rev=<u></u>163339&r1=163338&r2=163339&<u></u>view=diff</a><br>

==============================<u></u>==============================<u></u>==================<br>
--- llvm/trunk/lib/CodeGen/<u></u>SelectionDAG/<u></u>ScheduleDAGRRList.cpp (original)<br>
+++ llvm/trunk/lib/CodeGen/<u></u>SelectionDAG/<u></u>ScheduleDAGRRList.cpp Thu Sep  6 14:06:06 2012<br>
@@ -1758,6 +1758,7 @@<br>
      return V;<br>
    }<br>
<br>
+#ifndef NDEBUG<br>
    void dump(ScheduleDAG *DAG) const {<br>
      // Emulate pop() without clobbering NodeQueueIds.<br>
      std::vector<SUnit*> DumpQueue = Queue;<br>
@@ -1768,6 +1769,7 @@<br>
        SU->dump(DAG);<br>
      }<br>
    }<br>
+#endif<br>
  };<br>
<br>
  typedef RegReductionPriorityQueue<bu_<u></u>ls_rr_sort><br>
@@ -1895,6 +1897,7 @@<br>
  //===-------------------------<u></u>------------------------------<u></u>---------------===//<br>
<br>
  void RegReductionPQBase::<u></u>dumpRegPressure() const {<br>
+#ifndef NDEBUG<br>
    for (TargetRegisterInfo::regclass_<u></u>iterator I = TRI->regclass_begin(),<br>
           E = TRI->regclass_end(); I != E; ++I) {<br>
      const TargetRegisterClass *RC = *I;<br>
@@ -1904,6 +1907,7 @@<br>
      DEBUG(dbgs() << RC->getName() << ": " << RP << " / " << RegLimit[Id]<br>
            << '\n');<br>
    }<br>
+#endif<br>
  }<br>
<br>
  bool RegReductionPQBase::<u></u>HighRegPressure(const SUnit *SU) const {<br>
<br>
Modified: llvm/trunk/lib/CodeGen/<u></u>SelectionDAG/<u></u>ScheduleDAGSDNodes.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/SelectionDAG/ScheduleDAGSDNodes.cpp?rev=163339&r1=163338&r2=163339&view=diff" target="_blank">http://llvm.org/viewvc/llvm-<u></u>project/llvm/trunk/lib/<u></u>CodeGen/SelectionDAG/<u></u>ScheduleDAGSDNodes.cpp?rev=<u></u>163339&r1=163338&r2=163339&<u></u>view=diff</a><br>

==============================<u></u>==============================<u></u>==================<br>
--- llvm/trunk/lib/CodeGen/<u></u>SelectionDAG/<u></u>ScheduleDAGSDNodes.cpp (original)<br>
+++ llvm/trunk/lib/CodeGen/<u></u>SelectionDAG/<u></u>ScheduleDAGSDNodes.cpp Thu Sep  6 14:06:06 2012<br>
@@ -643,6 +643,7 @@<br>
  }<br>
<br>
  void ScheduleDAGSDNodes::dumpNode(<u></u>const SUnit *SU) const {<br>
+#ifndef NDEBUG<br>
    if (!SU->getNode()) {<br>
      dbgs() << "PHYS REG COPY\n";<br>
      return;<br>
@@ -659,8 +660,10 @@<br>
      dbgs() << "\n";<br>
      GluedNodes.pop_back();<br>
    }<br>
+#endif<br>
  }<br>
<br>
+#ifndef NDEBUG<br>
  void ScheduleDAGSDNodes::<u></u>dumpSchedule() const {<br>
    for (unsigned i = 0, e = Sequence.size(); i != e; i++) {<br>
      if (SUnit *SU = Sequence[i])<br>
@@ -669,6 +672,7 @@<br>
        dbgs() << "**** NOOP ****\n";<br>
    }<br>
  }<br>
+#endif<br>
<br>
  #ifndef NDEBUG<br>
  /// VerifyScheduledSequence - Verify that all SUnits were scheduled and that<br>
<br>
Modified: llvm/trunk/lib/CodeGen/<u></u>SlotIndexes.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/SlotIndexes.cpp?rev=163339&r1=163338&r2=163339&view=diff" target="_blank">http://llvm.org/viewvc/llvm-<u></u>project/llvm/trunk/lib/<u></u>CodeGen/SlotIndexes.cpp?rev=<u></u>163339&r1=163338&r2=163339&<u></u>view=diff</a><br>

==============================<u></u>==============================<u></u>==================<br>
--- llvm/trunk/lib/CodeGen/<u></u>SlotIndexes.cpp (original)<br>
+++ llvm/trunk/lib/CodeGen/<u></u>SlotIndexes.cpp Thu Sep  6 14:06:06 2012<br>
@@ -143,6 +143,7 @@<br>
  }<br>
<br>
<br>
+#ifndef NDEBUG<br>
  void SlotIndexes::dump() const {<br>
    for (IndexList::const_iterator itr = indexList.begin();<br>
         itr != indexList.end(); ++itr) {<br>
@@ -159,6 +160,7 @@<br>
      dbgs() << "BB#" << i << "\t[" << MBBRanges[i].first << ';'<br>
             << MBBRanges[i].second << ")\n";<br>
  }<br>
+#endif<br>
<br>
  // Print a SlotIndex to a raw_ostream.<br>
  void SlotIndex::print(raw_ostream &os) const {<br>
@@ -168,9 +170,11 @@<br>
      os << "invalid";<br>
  }<br>
<br>
+#ifndef NDEBUG<br>
  // Dump a SlotIndex to stderr.<br>
  void SlotIndex::dump() const {<br>
    print(dbgs());<br>
    dbgs() << "\n";<br>
  }<br>
+#endif<br>
<br>
<br>
Modified: llvm/trunk/lib/CodeGen/<u></u>SplitKit.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/SplitKit.cpp?rev=163339&r1=163338&r2=163339&view=diff" target="_blank">http://llvm.org/viewvc/llvm-<u></u>project/llvm/trunk/lib/<u></u>CodeGen/SplitKit.cpp?rev=<u></u>163339&r1=163338&r2=163339&<u></u>view=diff</a><br>

==============================<u></u>==============================<u></u>==================<br>
--- llvm/trunk/lib/CodeGen/<u></u>SplitKit.cpp (original)<br>
+++ llvm/trunk/lib/CodeGen/<u></u>SplitKit.cpp Thu Sep  6 14:06:06 2012<br>
@@ -356,6 +356,7 @@<br>
    Edit->anyRematerializable(0);<br>
  }<br>
<br>
+#ifndef NDEBUG<br>
  void SplitEditor::dump() const {<br>
    if (RegAssign.empty()) {<br>
      dbgs() << " empty\n";<br>
@@ -366,6 +367,7 @@<br>
      dbgs() << " [" << I.start() << ';' << I.stop() << "):" << I.value();<br>
    dbgs() << '\n';<br>
  }<br>
+#endif<br>
<br>
  VNInfo *SplitEditor::defValue(<u></u>unsigned RegIdx,<br>
                                const VNInfo *ParentVNI,<br>
<br>
Modified: llvm/trunk/lib/CodeGen/<u></u>VirtRegMap.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/VirtRegMap.cpp?rev=163339&r1=163338&r2=163339&view=diff" target="_blank">http://llvm.org/viewvc/llvm-<u></u>project/llvm/trunk/lib/<u></u>CodeGen/VirtRegMap.cpp?rev=<u></u>163339&r1=163338&r2=163339&<u></u>view=diff</a><br>

==============================<u></u>==============================<u></u>==================<br>
--- llvm/trunk/lib/CodeGen/<u></u>VirtRegMap.cpp (original)<br>
+++ llvm/trunk/lib/CodeGen/<u></u>VirtRegMap.cpp Thu Sep  6 14:06:06 2012<br>
@@ -126,9 +126,11 @@<br>
    OS << '\n';<br>
  }<br>
<br>
+#ifndef NDEBUG<br>
  void VirtRegMap::dump() const {<br>
    print(dbgs());<br>
  }<br>
+#endif<br>
<br>
  //===-------------------------<u></u>------------------------------<u></u>---------------===//<br>
  //                              VirtRegRewriter<br>
<br>
Modified: llvm/trunk/lib/Target/X86/<u></u>X86FloatingPoint.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/X86/X86FloatingPoint.cpp?rev=163339&r1=163338&r2=163339&view=diff" target="_blank">http://llvm.org/viewvc/llvm-<u></u>project/llvm/trunk/lib/Target/<u></u>X86/X86FloatingPoint.cpp?rev=<u></u>163339&r1=163338&r2=163339&<u></u>view=diff</a><br>

==============================<u></u>==============================<u></u>==================<br>
--- llvm/trunk/lib/Target/X86/<u></u>X86FloatingPoint.cpp (original)<br>
+++ llvm/trunk/lib/Target/X86/<u></u>X86FloatingPoint.cpp Thu Sep  6 14:06:06 2012<br>
@@ -171,6 +171,7 @@<br>
      // Shuffle live registers to match the expectations of successor blocks.<br>
      void finishBlockStack();<br>
<br>
+#ifndef NDEBUG<br>
      void dumpStack() const {<br>
        dbgs() << "Stack contents:";<br>
        for (unsigned i = 0; i != StackTop; ++i) {<br>
@@ -181,6 +182,7 @@<br>
          dbgs() << ", ST" << i << " in FP" << unsigned(PendingST[i]);<br>
        dbgs() << "\n";<br>
      }<br>
+#endif<br>
<br>
      /// getSlot - Return the stack slot number a particular register number is<br>
      /// in.<br>
<br>
Modified: llvm/trunk/lib/Target/X86/<u></u>X86ISelDAGToDAG.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/X86/X86ISelDAGToDAG.cpp?rev=163339&r1=163338&r2=163339&view=diff" target="_blank">http://llvm.org/viewvc/llvm-<u></u>project/llvm/trunk/lib/Target/<u></u>X86/X86ISelDAGToDAG.cpp?rev=<u></u>163339&r1=163338&r2=163339&<u></u>view=diff</a><br>

==============================<u></u>==============================<u></u>==================<br>
--- llvm/trunk/lib/Target/X86/<u></u>X86ISelDAGToDAG.cpp (original)<br>
+++ llvm/trunk/lib/Target/X86/<u></u>X86ISelDAGToDAG.cpp Thu Sep  6 14:06:06 2012<br>
@@ -100,6 +100,7 @@<br>
        Base_Reg = Reg;<br>
      }<br>
<br>
+#ifndef NDEBUG<br>
      void dump() {<br>
        dbgs() << "X86ISelAddressMode " << this << '\n';<br>
        dbgs() << "Base_Reg ";<br>
@@ -133,6 +134,7 @@<br>
          dbgs() << "nul";<br>
        dbgs() << " JT" << JT << " Align" << Align << '\n';<br>
      }<br>
+#endif<br>
    };<br>
  }<br>
<br>
<br>
<br>
______________________________<u></u>_________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/<u></u>mailman/listinfo/llvm-commits</a><br>
<br>
</blockquote>
<br>
______________________________<u></u>_________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/<u></u>mailman/listinfo/llvm-commits</a><br>
</blockquote></div><br><br clear="all"><br>-- <br>~Craig<br>