<div dir="ltr">Hi Silviu,<div><br></div><div>Indeed - my testing showed that the branched version was no faster than the csel version on an OoO core anyway, so I was not advocating the branched solution.</div><div><br></div>
<div>James</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On 10 July 2014 11:26, Silviu Baranga <span dir="ltr"><<a href="mailto:silviu.baranga@gmail.com" target="_blank">silviu.baranga@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">James,<br>
<br>
It seems to me that the branch mispredict cost for the case where the values of X are random would outweigh the benefits of this transformation for your alternative code sequence, even on OoO cores.<br>
<br>
I don't think it would entirely ok to make that assumption here (X >= 0 predictable).<br>
<br>
This point obviously doesn't matter for the csel solution.<br>
<br>
Thanks,<br>
Silviu<br>
<div class="HOEnZb"><div class="h5"><br>
<a href="http://reviews.llvm.org/D4438" target="_blank">http://reviews.llvm.org/D4438</a><br>
<br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div><br></div>