Re: [PATCH] S390: Fix vec_load_bndry.

2015-08-20 Thread Dominik Vogt
...@linux.vnet.ibm.com Date: Tue, 18 Aug 2015 13:11:08 +0100 Subject: [PATCH] S390: Fix vec_load_bndry. In one place it required 64, 128, ..., 4096 as the second argument and in another place it required that value to fit into three bits. --- gcc/config/s390/s390-builtins.def | 18

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 toolchain

[PATCH] S390: Fix vec_load_bndry.

2015-08-18 Thread Dominik Vogt
2001 From: Dominik Vogt v...@linux.vnet.ibm.com Date: Tue, 18 Aug 2015 13:11:08 +0100 Subject: [PATCH] S390: Fix vec_load_bndry. In one place it required 64, 128, ..., 4096 as the second argument and in another place it required that value to fit into three bits. --- gcc/config/s390/s390

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