This is an archive of the discontinued LLVM Phabricator instance.

Differential D10072

Fix debug printing in RegisterPressure.cpp
ClosedPublic

Authored by escha on May 27 2015, 2:41 PM.

Details

Reviewers
atrick
escha
resistor
Summary

RegisterPressure tracks physical registers by the RegUnit, so it was printing incorrect results for live physical registers.

Diff Detail

Repository
rL LLVM

Event Timeline

escha updated this revision to Diff 26635.May 27 2015, 2:41 PM
escha retitled this revision from to Fix debug printing in RegisterPressure.cpp.
escha updated this object.
escha edited the test plan for this revision. (Show Details)
escha added reviewers: resistor, atrick.
escha set the repository for this revision to rL LLVM.
escha added a subscriber: Unknown Object (MLST).
atrick edited edge metadata.May 27 2015, 2:47 PM

That looks like it’s working. Thanks for the fix!

Andy

escha accepted this revision.May 27 2015, 3:55 PM
escha added a reviewer: escha.
This revision is now accepted and ready to land.May 27 2015, 3:55 PM
escha closed this revision.May 29 2015, 10:22 AM

Revision Contents

PathSize
lib/
CodeGen/
4 lines

Diff 26635

lib/CodeGen/RegisterPressure.cpp

Context not available.
dumpRegSetPressure(MaxSetPressure, TRI); dumpRegSetPressure(MaxSetPressure, TRI);
dbgs() << "Live In: "; dbgs() << "Live In: ";
for (unsigned i = 0, e = LiveInRegs.size(); i < e; ++i) for (unsigned i = 0, e = LiveInRegs.size(); i < e; ++i)
dbgs() << PrintReg(LiveInRegs[i], TRI) << " "; dbgs() << PrintVRegOrUnit(LiveInRegs[i], TRI) << " ";
dbgs() << '\n'; dbgs() << '\n';
dbgs() << "Live Out: "; dbgs() << "Live Out: ";
for (unsigned i = 0, e = LiveOutRegs.size(); i < e; ++i) for (unsigned i = 0, e = LiveOutRegs.size(); i < e; ++i)
dbgs() << PrintReg(LiveOutRegs[i], TRI) << " "; dbgs() << PrintVRegOrUnit(LiveOutRegs[i], TRI) << " ";
dbgs() << '\n'; dbgs() << '\n';
} }
Context not available.