R600: Expand vselect and SRA for v2i32 and v4i32

Tom Stellard tom at stellard.net
Mon May 6 19:04:30 PDT 2013


On Mon, May 06, 2013 at 07:35:42PM -0500, Aaron Watry wrote:
> These two patches fix a number of piglit OpenCL test failures on my
> HD6850 (Barts).
> 
> There are no piglit CL test regressions and the llvm make check runs
> without any unexpected failures.
>

Hi Aaron,

These patches look good to me, but you should also add some lit tests
for these opcodes.  For R600, the tests are located in
test/CodeGen/R600.  A good example to look at is the srl.ll test, the
sra.ll test should be more or less the same.  fcmp-cnd.ll would
be a good example for the vselect.ll test as a vector version of it
should generate vselect nodes (though you'll want to make sure to use
i32 types instead of floats).

-Tom



More information about the llvm-commits mailing list