<div dir="ltr">Hi Tim,<div><br></div><div>I added some CodeGen test and try to use clang-format to check code style. I guess there will be less code format problem in future:)</div><div>Please review,thanks.</div></div><div class="gmail_extra">
<br><br><div class="gmail_quote">2013/11/1 Tim Northover <span dir="ltr"><<a href="mailto:t.p.northover@gmail.com" target="_blank">t.p.northover@gmail.com</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi Kevin,<br>
<br>
The Clang patch looks good, and as far as I can see the LLVM one is OK<br>
except that it needs CodeGen tests.<br>
<br>
Oh, and the spacing on this line and the int version is wonky:<br>
+        EltVT = (EltSize == 64)? MVT::f64: MVT::f32;<br>
<br>
Cheers.<br>
<span class="HOEnZb"><font color="#888888"><br>
Tim.<br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br><div dir="ltr">Best Regards,<div><br></div><div>Kevin Qin</div></div>
</div>