<div dir="ltr">Yeah, the emacs one does as well, but I've not yet figured out a way to make gmail do code highlighting. :)<div><br></div><div>-eric<br><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Mar 14, 2013 at 2:32 PM, Arnold Schwaighofer <span dir="ltr"><<a href="mailto:aschwaighofer@apple.com" target="_blank">aschwaighofer@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Sure. I will insert some white space and remove the attributes. The vim .ll syntax mode makes comments stick out so I did not notice this.<br>

<br>
r177114<br>
<div class="HOEnZb"><div class="h5"><br>
On Mar 14, 2013, at 4:20 PM, Eric Christopher <<a href="mailto:echristo@gmail.com">echristo@gmail.com</a>> wrote:<br>
<br>
> A couple of small nits here:<br>
><br>
> +<br>
> +; CHECK: Checking a loop in "scalarselect"<br>
> +define void @scalarselect(i1 %cond) nounwind uwtable ssp {<br>
><br>
> Don't really need the uwtable and ssp attributes here.<br>
><br>
> +; 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>
><br>
> 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...<br>

><br>
> -eric<br>
<br>
</div></div></blockquote></div><br></div></div></div>