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.weig...@de.ibm.com

Reply via email to