<div dir="ltr">Why does it block that?</div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Feb 7, 2017 at 1:19 PM, Alexey Bataev <span dir="ltr"><<a href="mailto:a.bataev@hotmail.com" target="_blank">a.bataev@hotmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">The first option is not suitable, it blocks min/max reduction vectorization.<br>
<br>
Best regards,<br>
Alexey Bataev<br>
<br>
> 8 февр. 2017 г., в 0:09, Michael Kuperstein via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> написал(а):<br>
<div class="HOEnZb"><div class="h5">><br>
> mkuper added a comment.<br>
><br>
>> What should I do then?<br>
><br>
> Short term - maybe nothing?<br>
> Is this patch blocking anything? I understand this is part of the work to support min/max reductions, but why is it necessary? Can we go forward with that without regressing any existing cases?<br>
><br>
> Longer term - it would probably be good to try to come up with a saner, or at least, more principled way to do root selection, that also doesn't cause us to look at instructions several times. I don't think adding more ad-hoc cases (CallInst) is the way to go. I'm fairly sure we can come up with other examples like this.<br>
><br>
><br>
> <a href="https://reviews.llvm.org/D29449" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D29449</a><br>
><br>
><br>
><br>
</div></div></blockquote></div><br></div>