
<html><head><meta http-equiv="Content-Type" content="text/html charset=iso-8859-1"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;">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>On Aug 2, 2013, at 5:22 AM, Renato Golin <<a href="mailto:renato.golin@linaro.org">renato.golin@linaro.org</a>> wrote:</div><br class="Apple-interchange-newline"><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; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;"><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><span><armtt.patch></span></div></blockquote></div><br></div></body></html>