This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] S390: Fix vec_load_bndry.
- From: "Ulrich Weigand" <uweigand at de dot ibm dot com>
- To: vogt at linux dot vnet dot ibm dot com
- Cc: gcc-patches at gcc dot gnu dot org, Ulrich dot Weigand at de dot ibm dot com (Ulrich Weigand), krebbel at linux dot vnet dot ibm dot com (Andreas Krebbel)
- Date: Tue, 18 Aug 2015 14:49:30 +0200 (CEST)
- Subject: Re: [PATCH] S390: Fix vec_load_bndry.
- Authentication-results: sourceware.org; auth=none
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 correct values and generate correct assembler ...
> gcc/ChangeLog
>
> * config/s390/s390-builtins.def: S390: Fix value range of
> vec_load_bndry.
The "S390: " seems out of place here.
Thanks,
Ulrich
--
Dr. Ulrich Weigand
GNU/Linux compilers and toolchain
Ulrich.Weigand@de.ibm.com