I intend to look into fixing that. There's likely something missing during DAG combine that should be able to fix this up.<br><br><div class="gmail_quote">On Thu, Apr 12, 2012 at 3:23 AM, Florian Pflug <span dir="ltr"><<a href="mailto:fgp@phlo.org">fgp@phlo.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Apr12, 2012, at 08:38 , Craig Topper wrote:<br>
> Interestingly, the AVX ptest intrinsics are correctly taking 4 x i64 arguments. I'll fix the 128-bit versions to take 2 x i64.<br>
<br>
</div>Wouldn't it also make more sense for the PTEST intrinsics to return an i1, not an i32? One could then use the results of<br>
llvm.x86.sse41.ptest{z,c,nc} directly as a condition for br, without the intermediate icmp step. Or so I imagine, at least.<br>
<br>
best regards,<br>
Florian Pflug<br>
<br>
</blockquote></div><br><br clear="all"><br>-- <br>~Craig<br>