[RegAllocGreedy] debug output fix

Jonas Paulsson via llvm-commits llvm-commits at lists.llvm.org
Fri May 18 02:40:24 PDT 2018


Hi,

It seems like

LLVM_DEBUG(dbgs() << "should evict: " << B << " w= " << B.weight << '\n');

is duplicating the printing of weight, since LiveInterval already prints it:

void LiveInterval::print(raw_ostream &OS) const {
   OS << printReg(reg) << ' ';
   super::print(OS);
   // Print subranges
   for (const SubRange &SR : subranges())
     OS << SR;
   OS << " weight:" << weight;
}

I see this in the dumps:
selectOrSplit ADDR64Bit:%96 [32r,3712r:0)  0 at 32r weight:1.609069e-02 
w=1.609069e-02
--

Is this needed in special situations, or is my patch ok?

/Jonas


-------------- next part --------------
A non-text attachment was scrubbed...
Name: latest.patch
Type: text/x-patch
Size: 486 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180518/0694ee87/attachment.bin>


More information about the llvm-commits mailing list