
<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><div>On Aug 17, 2010, at 2:11 PM, Jakob Stoklund Olesen wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div><br>On Aug 17, 2010, at 2:00 PM, Evan Cheng wrote:<br><br><blockquote type="cite">Author: evancheng<br></blockquote><blockquote type="cite">Date: Tue Aug 17 16:00:37 2010<br></blockquote><blockquote type="cite">New Revision: 111285<br></blockquote><blockquote type="cite"><br></blockquote><blockquote type="cite">URL: <a href="http://llvm.org/viewvc/llvm-project?rev=111285&view=rev">http://llvm.org/viewvc/llvm-project?rev=111285&view=rev</a><br></blockquote><blockquote type="cite">Log:<br></blockquote><blockquote type="cite">PHI elimination shouldn't require machineloopinfo since it's used at -O0. Move the requirement to LiveIntervalAnalysis instead. Note this does not change the number of times machineloopinfo is computed.<br></blockquote><br>Thanks!<br><br>Note that both SimpleRegisterCoalescing and CalcSpillWeights require LoopInfo, so you may be able to get away with only AU.addPreserved<MachineLoopInfo>() in LiveIntervalAnalysis.<br></div></blockquote><div><br></div>That's what I thought at first but it didn't work as expected.</div><div><br><blockquote type="cite"><div><br>Does this mean that we are only calculating LoopInfo once now? That's awesome.<font class="Apple-style-span" color="#000000"><font class="Apple-style-span" color="#144FAE"><br></font></font></div></blockquote><br></div><div>No. Tail duplication and if-conversion are not preserving it.</div><div><br></div><div>Evan</div><div><br></div></body></html>