Also you should fix the two other places that do the same thing as well. X86AsmPrinter.cpp and X86IntelInstPrinter.cpp.<br><br><div class="gmail_quote">On Sun, Feb 5, 2012 at 9:24 PM, Craig Topper <span dir="ltr"><<a href="mailto:craig.topper@gmail.com">craig.topper@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">As nothing protects the value written in the intrinsic in the source code. I don't think that having a default case for all unexpected values is the right way to go. Maybe the immediate should be masked to remove the reserved encodings? Or the full value should be printed as an integer if its too big?<div class="HOEnZb">
<div class="h5"><br>
<br><div class="gmail_quote">On Sun, Feb 5, 2012 at 5:01 PM, Eric Christopher <span dir="ltr"><<a href="mailto:echristo@apple.com" target="_blank">echristo@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Looks fine, but could you get an exhaustive testcase please?<br>
<br>
vcmpgeps reg1, reg2, reg3<br>
vcmapfalseps reg1, reg2, reg3<br>
<br>
etc?<br>
<br>
Thanks!<br>
<div><div><br>
On Feb 5, 2012, at 5:12 AM, Demikhovsky, Elena wrote:<br>
<br>
> This IR code<br>
> %res = call <8 x float> @llvm.x86.avx.cmp.ps.256(<8 x float> %a0, <8 x float> %a1, i8 14)<br>
> fails with assertion:<br>
><br>
> llc: X86ATTInstPrinter.cpp:62: void llvm::X86ATTInstPrinter::printSSECC(const llvm::MCInst*, unsigned int, llvm::raw_ostream&): Assertion `0 && "Invalid ssecc argument!"' failed.<br>
> 0  llc             0x0000000001355803<br>
> 1  llc             0x0000000001355dc9<br>
> 2  libpthread.so.0 0x00007f79a30575d0<br>
> 3  libc.so.6       0x00007f79a23a1945 gsignal + 53<br>
> 4  libc.so.6       0x00007f79a23a2f21 abort + 385<br>
> 5  libc.so.6       0x00007f79a239a810 __assert_fail + 240<br>
> 6  llc             0x00000000011858d5 llvm::X86ATTInstPrinter::printSSECC(llvm::MCInst const*, unsigned int, llvm::raw_ostream&) + 119<br>
><br>
><br>
> Attached a fix. Please review.<br>
><br>
><br>
> - Elena<br>
><br>
><br>
> ---------------------------------------------------------------------<br>
> Intel Israel (74) Limited<br>
><br>
> This e-mail and any attachments may contain confidential material for<br>
> the sole use of the intended recipient(s). Any review or distribution<br>
> by others is strictly prohibited. If you are not the intended<br>
> recipient, please contact the sender and delete all copies.<br>
</div></div>> <cmp.diff>_______________________________________________<br>
> llvm-commits mailing list<br>
> <a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">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>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">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>
</blockquote></div><br><br clear="all"><br></div></div><span class="HOEnZb"><font color="#888888">-- <br>~Craig<br>
</font></span></blockquote></div><br><br clear="all"><br>-- <br>~Craig<br>