<div dir="ltr"><div style>A couple of small nits here:</div><br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
+<br>
+; CHECK: Checking a loop in "scalarselect"<br>
+define void @scalarselect(i1 %cond) nounwind uwtable ssp {<br>
</blockquote><div><br></div><div style>Don't really need the uwtable and ssp attributes here.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
+; A scalar select has a cost of 1 on core2<br>
+; CHECK: cost of 1 for VF 2 {{.*}}  select i1 %cond, i32 %6, i32 0<br>
+  %sel = select i1 %cond, i32 %6, i32 zeroinitializer<br></blockquote><div><br></div><div style>The checks mid-block are _really_ hard to see. Mind hoisting or providing some vertical whitespace, or something? If you hoist you may want to comment the particular select below that you're checking on as well, but...</div>
<div style><br></div><div style>-eric </div></div></div></div>