<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>