
<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><div></div><div><br></div><div><div>On Aug 13, 2013, at 4:22 PM, Jakob Stoklund Olesen <<a href="mailto:stoklund@2pi.dk">stoklund@2pi.dk</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div style="font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;"><br>On Aug 13, 2013, at 3:57 PM, Matthias Braun <<a href="mailto:mbraun@apple.com">mbraun@apple.com</a>> wrote:<br><br><blockquote type="cite">MatzeB added you to the CC list for the revision "some cleanups in regalloc code".<br><br>Just some small cleanups of stuff I spotted while reading register allocation<br>code:<br><br>remove unused struct/enum<br><br>LiveIntervalUnion is not used in RegAllocBase<br><br>remove declaration of nonexistant function<br><br>improve misleading comment<br><br>remove unused function<br><br>remove unnecessary parameter to RenumberValues<br><br><a href="http://llvm-reviews.chandlerc.com/D1393">http://llvm-reviews.chandlerc.com/D1393</a><br><br>Files:<br>include/llvm/CodeGen/LiveInterval.h<br>lib/CodeGen/LiveInterval.cpp<br>lib/CodeGen/LiveIntervalAnalysis.cpp<br>lib/CodeGen/LiveRangeCalc.h<br>lib/CodeGen/LiveRangeEdit.cpp<br>lib/CodeGen/RegAllocBase.h<br>lib/CodeGen/SplitKit.cpp<br></blockquote><br>Hi Matthias,<br><br>This all looks good to me, but it should be broken into separate commits.<br><br>Thanks,<br>/jakob<br><br>_______________________________________________<br>llvm-commits mailing list<br><a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br><a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a></div></blockquote></div><br></body></html>