I think Arnold intended for the test case to be added to the changelist. Such a test would go into the test/CodeGen/ARM subdirectory and follow the standard practices (i.e. CHECK annotations for the expected output, etc.).<div>
<br></div><div>Steve</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Aug 14, 2012 at 3:55 PM, Yin Ma <span dir="ltr"><<a href="mailto:yinma@codeaurora.org" target="_blank">yinma@codeaurora.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Arnold,<br>
<br>
    Here we go. This is the test case.<br>
<span class="HOEnZb"><font color="#888888"><br>
              Yin<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
-----Original Message-----<br>
From: Arnold Schwaighofer [mailto:<a href="mailto:arnolds@codeaurora.org">arnolds@codeaurora.org</a>]<br>
Sent: Tuesday, August 14, 2012 2:17 PM<br>
To: Yin Ma<br>
Cc: <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
Subject: Re: [llvm-commits] [PATCH][Please review] Enable Vector Select for<br>
ARM<br>
<br>
Can you add a test case?<br>
<br>
-    // Since only 64-bit and 128-bit vectors are legal on ARM and<br>
-    // we've eliminated the other cases...<br>
-    assert(SourceVecs[i].getValueType().getVectorNumElements() ==<br>
2*NumElts &&<br>
-           "unexpected vector sizes in ReconstructShuffle");<br>
<br>
The comments above the removed assert look suspicious. They seem to indicate<br>
that this case was supposed to be handle further up/else where?<br>
<br>
Best,<br>
Arnold<br>
<br>
On 8/14/2012 1:52 PM, Yin Ma wrote:<br>
> Hi,<br>
><br>
> So far vector select, such as  a < b ? a : c, when IR is generated for<br>
> vector<br>
><br>
> select on ARM. The ARM backend will trigger an assertion. The patch<br>
> will<br>
><br>
> add a handler code to make assertion go away.<br>
><br>
>     Please give a review again.<br>
><br>
> Thanks,<br>
><br>
>                 Yin<br>
><br>
><br>
><br>
> _______________________________________________<br>
> llvm-commits mailing list<br>
> <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
><br>
<br>
<br>
<br>
--<br>
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum.<br>
</div></div><br>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div><br></div>