<div dir="ltr">We spoke on IRC, the attached patch is a much better fix for now.<div><br></div><div>cheers,</div><div>--renato</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On 2 August 2013 17:14, Nadav Rotem <span dir="ltr"><<a href="mailto:nrotem@apple.com" target="_blank">nrotem@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word">Renato, <div><br></div><div>Excellent. Thanks for the quick analysis and fix. If Arnold gives a thumb up then we can commit this. </div>
<div><br></div><div>Thanks,</div><div>Nadav</div><div><br><div><div><div class="h5"><div>On Aug 2, 2013, at 5:22 AM, Renato Golin <<a href="mailto:renato.golin@linaro.org" target="_blank">renato.golin@linaro.org</a>> wrote:</div>
<br></div></div><blockquote type="cite"><div style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant:normal;font-weight:normal;letter-spacing:normal;line-height:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px">
<div><div class="h5"><div dir="ltr"><div>Nadav,</div><div><br></div><div>This patch fixes *ALL* test-suite problems! :D</div><div><br></div><div>Description:</div><div><br></div><div>This patch fixes the multiple breakages on ARM test-suite after the SLP<br>
</div><div>vectorizer was introduced by default on O3. The problem was a missing</div><div>legalization of the vector types on ARMTTI::getCmpSelInstrCost(). We</div><div>might have to review all cost functions for the proper legalization</div>
<div>of types to avoid problems like these in the future.</div><div><br></div><div>The cost of boolean vector comparison changed radically due to this patch,<br></div><div>which may be better (or not), depending on how the back-end will lower</div>
<div>these selects. Needs further investigation as well.</div><div><br></div><div>Arnold,</div><div><br></div><div>If the new costs are totally wrong, we might need further impediments and hard-code high costs on getCmpSelInstrCost(). Let me know if it rings a bell.</div>
<div><br></div><div>cheers,</div><div>--renato</div></div></div></div><span><armtt.patch></span></div></blockquote></div><br></div></div></blockquote></div><br></div>