<div dir="ltr">I had to revert it as it made 3 tests fail (ppc, arm). I will investigate the failure when I will be back home --- I have to catch my plane now :). Although there was no functional change intended, it changed register allocation on those targets. Removing the CalculateSpillWeight pass from the pipeline somehow affected the results of the pass pipeline on those targets. May be those tests' checks just need to be fine tuned to be immune to register allocation changes.<div>
<br></div><div>Cheers,</div><div>--</div><div>Arnaud</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Nov 8, 2013 at 7:05 PM, Arnaud Allard de Grandmaison <span dir="ltr"><<a href="mailto:arnaud.adegm@gmail.com" target="_blank">arnaud.adegm@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Committed at r194269<br><div><br></div><div>Thanks Jakob !</div><div><br></div><div>Cheers,</div><div>--</div>
<div>Arnaud</div></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Nov 8, 2013 at 6:34 PM, Jakob Stoklund Olesen <span dir="ltr"><<a href="mailto:stoklund@2pi.dk" target="_blank">stoklund@2pi.dk</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div><br>
On Nov 8, 2013, at 7:40 AM, Arnaud Allard de Grandmaison <<a href="mailto:arnaud.adegm@gmail.com" target="_blank">arnaud.adegm@gmail.com</a>> wrote:<br>
<br>
> Based on discussions with Lang Hames and Jakob Stoklund Olesen at the hacker's lab, and in the light of upcoming work on the PBQP register allocator, it was though that CalcSpillWeights does not need to be a pass. This change will enable to customize / tune the spill weight computation depending on the allocator.<br>


><br>
> Update the documentation style while there.<br>
><br>
> I also intend to take VirtRegAuxInfo out of line in a later patch as this is really a helper class for spill weight (re) computation.<br>
><br>
> Cheers,<br>
> --<br>
> Arnaud<br>
</div></div>> <0001-CalculateSpillWeights-does-not-need-to-be-a-pass.patch><br>
<br>
LGTM.<br>
<br>
Thanks,<br>
/jakob<br>
<br>
</blockquote></div><br></div>
</div></div></blockquote></div><br></div>