<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Dec 17, 2014 at 1:14 AM, Suyog Kamal Sarda <span dir="ltr"><<a href="mailto:suyog.sarda@samsung.com" target="_blank">suyog.sarda@samsung.com</a>></span> wrote:<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div id=":977" class="a3s" style="overflow:hidden">Hi Chandler,<br>
<br>
Thanks for pointing out the error. I agree to it and will shortly revert the revision.<br>
<br>
I have identified this and tried to address this issue - only do this for 'integer' type in<br>
<a href="http://reviews.llvm.org/D6675" target="_blank">http://reviews.llvm.org/D6675</a> (Some minor checking changes required here also.)<br>
<br>
Do you agree with <a href="http://reviews.llvm.org/D6675" target="_blank">http://reviews.llvm.org/D6675</a>  ?</div></blockquote></div><br>I'm not an expert in the SLP vectorizer, and not likely to be able to tell you what the right way to check for these things are. You should get a detailed review from someone who at least knows that code.</div><div class="gmail_extra"><br></div><div class="gmail_extra">All I can tell you is that the basic premise for when this is allowed should be:</div><div class="gmail_extra"><br></div><div class="gmail_extra">1) when we have integer types (add vs. fadd)</div><div class="gmail_extra">2) when the fadd is marked with fast math flags that allow reassociation (the 'fast' flag)</div><div class="gmail_extra"><br></div><div class="gmail_extra">It would be good to support this for floating point types when we have fast math flags. =] That case does come up, and this is a huge win in those cases.</div><div class="gmail_extra"><br></div><div class="gmail_extra">-Chandler</div></div>