[PATCH] D13857: Fix mapping of @llvm.arm.ssat/usat intrinsics to ssat/usat instructions

[Asiri Rathnayake via llvm-commits]

rmaprath added a comment.

In http://reviews.llvm.org/D13857#270039, @rengolin wrote:

> In http://reviews.llvm.org/D13857#270037, @rmaprath wrote:
>
> > /work/oss/git/build/./bin/llc -O1 -mtriple=armv6-none-none-eabi /work/oss/git/src/test/CodeGen/ARM/sat-arith.ll -o - | /work/oss/git/build/./bin/FileCheck /work/oss/git/src/test/CodeGen/ARM/sat-arith.ll
>
>
> You have to separate the good tests from the bad ones in different files. The good ones, you do just like the one you have (but expand to the limits of the ranges accepted). The bad ones, just replace:
>
>   ; RUN: llc ...
>   
>
> for
>
>   ; RUN: not llc ...
>   
>
> and you'll be able to use FileCheck to parse the error messages.
>
> There are many tests in CodeGen that follow this pattern.


Thanks! I'll add the tests.


Repository:
  rL LLVM

http://reviews.llvm.org/D13857