Commited r178208.<br><br>Thank you for the review.<br><br><div class="gmail_quote">On Wed, Mar 27, 2013 at 5:53 PM, Nadav Rotem <span dir="ltr"><<a href="mailto:nrotem@apple.com" target="_blank">nrotem@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">The change looks okay.  Can you please format the patch and the test case in a single file and use the FileCheck tool for testing the test case ?<div>
<br></div><div>Thanks,</div><div>Nadav<div><div class="h5"><br><br><span>On 03/27/13, <b>Akira Hatanaka </b> <<a href="mailto:ahatanak@gmail.com" target="_blank">ahatanak@gmail.com</a>> wrote:</span><blockquote style="border-left:1px solid #00f;padding-left:13px;margin-left:0" type="cite">
<div>The attached patch adds code in InstCombiner::FoldSelectOpOp that checks if a Type is a vector before calling function Type::getVectorNumElements.<br>Without this fix, the following command asserts:<br><br>$ opt instcombine1.ll -o instcombine1.bc -O3<br>


<br>Please review.<br>
</div></blockquote></div></div></div>
</blockquote></div><br>