<div dir="ltr">Hi,<div><br></div><div>I was in two minds about this. I thought like you did originally, but I ended up deciding that trying to get the extractelements inside the predicated blocks would be a false economy. Sinking those extracts is something InstCombine or similar should do trivially (in fact, that would be a good test - I'll add that).</div><div><br></div><div>James</div></div><br><div class="gmail_quote"><div dir="ltr">On Tue, 1 Sep 2015 at 20:30 Michael Zolotukhin via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">mzolotukhin added a comment.<br>
<br>
Hi James,<br>
<br>
While the general idea looks appealing to me, I'm worried on the changes that we have in the test. It looks like we increased loop's critical path length by this change by moving `extractelement` up from branches. What would it take to change it back?<br>
<br>
Thanks,<br>
Michael<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="http://reviews.llvm.org/D12480" rel="noreferrer" target="_blank">http://reviews.llvm.org/D12480</a><br>
<br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</blockquote></div>