OpenE2K
/
gcc
2
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

optabs.c (expand_binop): Don't create a broadcast vector with a source element wider than the inner mode. 

[gcc]

2015-09-03  Bill Schmidt  <wschmidt@linux.vnet.ibm.com>

	* optabs.c (expand_binop): Don't create a broadcast vector with a
	source element wider than the inner mode.

[gcc/testsuite]

2015-09-03  Bill Schmidt  <wschmidt@linux.vnet.ibm.com>

	* gcc.target/powerpc/vec-shift.c: New test.

From-SVN: r227444
This commit is contained in:
Bill Schmidt 2015-09-03 13:52:17 +00:00 committed by William Schmidt
parent 4413ddbcb2
commit 5179d2a213
4 changed files with 38 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
gcc/ChangeLog
View File

@ -1,3 +1,8 @@
2015-09-03 Bill Schmidt <wschmidt@linux.vnet.ibm.com>
* optabs.c (expand_binop): Don't create a broadcast vector with a
source element wider than the inner mode.
2015-09-03 Richard Biener <rguenther@suse.de>
* varasm.c (output_constant): Use fold_convert instead of

9
gcc/optabs.c
View File

@ -1608,6 +1608,15 @@ expand_binop (machine_mode mode, optab binoptab, rtx op0, rtx op1,
if (otheroptab && optab_handler (otheroptab, mode) != CODE_FOR_nothing)
{
/* The scalar may have been extended to be too wide. Truncate
it back to the proper size to fit in the broadcast vector. */
machine_mode inner_mode = GET_MODE_INNER (mode);
if (!CONST_INT_P (op1)
&& (GET_MODE_BITSIZE (inner_mode)
< GET_MODE_BITSIZE (GET_MODE (op1))))
op1 = force_reg (inner_mode,
simplify_gen_unary (TRUNCATE, inner_mode, op1,
GET_MODE (op1)));
rtx vop1 = expand_vector_broadcast (mode, op1);
if (vop1)
{

4
gcc/testsuite/ChangeLog
View File

@ -1,3 +1,7 @@
2015-09-03 Bill Schmidt <wschmidt@linux.vnet.ibm.com>
* gcc.target/powerpc/vec-shift.c: New test.
2015-09-03 Tom de Vries <tom@codesourcery.com>
PR tree-optimization/65637

20
gcc/testsuite/gcc.target/powerpc/vec-shift.c Normal file
View File

@ -0,0 +1,20 @@
/* { dg-do compile { target { powerpc*-*-* } } } */
/* { dg-require-effective-target powerpc_altivec_ok } */
/* { dg-skip-if "" { powerpc*-*-darwin* } { "*" } { "" } } */
/* { dg-skip-if "do not override -mcpu" { powerpc*-*-* } { "-mcpu=*" } { "-mcpu=power7" } } */
/* { dg-options "-mcpu=power7 -O2" } */
/* This used to ICE. During gimplification, "i" is widened to an unsigned
int. We used to fail at expand time as we tried to cram an SImode item
into a QImode memory slot. This has been fixed to properly truncate the
shift amount when splatting it into a vector. */
typedef unsigned char v16ui __attribute__((vector_size(16)));
v16ui vslb(v16ui v, unsigned char i)
{
return v << i;
}
/* { dg-final { scan-assembler "vspltb" } } */
/* { dg-final { scan-assembler "vslb" } } */