Re: [PATCH] S390: Fix vec_load_bndry.

2015-08-20 Thread Ulrich Weigand
Dominik Vogt wrote: > gcc/ChangeLog > > * config/s390/s390-builtins.def: Fix value range of vec_load_bndry. > > gcc/testsuite/ChangeLog > > * gcc.target/s390/zvector/vec-load_bndry-1.c: New test. This is OK. Thanks, Ulrich -- Dr. Ulrich Weigand GNU/Linux compilers and toolch

Re: [PATCH] S390: Fix vec_load_bndry.

2015-08-20 Thread Dominik Vogt
On Tue, Aug 18, 2015 at 02:49:30PM +0200, Ulrich Weigand wrote: > Dominik Vogt wrote: > > > The attached patch fixes the vec_load_bndry builtin on S390. The second > > argument must be one of 64, 128, 256, ..., 4096, but the table expected that > > value to fit into 3 bits. > > Makes sense. How

Re: [PATCH] S390: Fix vec_load_bndry.

2015-08-18 Thread Ulrich Weigand
Dominik Vogt wrote: > The attached patch fixes the vec_load_bndry builtin on S390. The second > argument must be one of 64, 128, 256, ..., 4096, but the table expected that > value to fit into 3 bits. Makes sense. However, I'd really like to see a testcase that verifies we're accepting the corr