<div dir="ltr">Hi Chad, Timur<div><br></div><div>Just wanted to make a point here.<br><div><br></div><div>This patch actually eased further transformations like SLP Vectoization.</div><div><br></div><div>Test case :</div><div><br></div><div>sum = a[0] + a[1] + a[2] + a[3]</div><div><br></div><div>Earlier, without this patch, re-associate pass would re-organize this expression into<br></div><div><br></div><div>sum = (((a[1] + a[0]) + a[2]) + a[3]) </div><div><br></div><div>which made SLP vectorizer difficult to identify this type of horizontal reduction.</div><div>With this patch, the expression would remain<br></div><div><br></div><div>sum = (((a[0] + a[1]) + a[2]) + a[3]) </div><div><br></div><div>which made SLP vectorizer task simpler.<br></div><div><br></div><div>Patch holds good value for further optimization passes, </div><div>though as Timur reports that its breaking some test cases.</div><div><br></div><div>-- <br><div class="gmail_signature">With regards,<br>Suyog Sarda</div></div></div></div>