git
/
gcc
mirror of git://gcc.gnu.org/git/gcc.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

re PR tree-optimization/53748 (ice in fold_convert_loc) 

PR tree-optimization/53748
	* tree-ssa-phiopt.c (conditional_replacement): Only optimize
	if arg0/arg1 have integral or pointer types.

	* gcc.c-torture/compile/pr53748.c: New test.

From-SVN: r188975
This commit is contained in:
Jakub Jelinek 2012-06-26 10:02:20 +02:00 committed by Jakub Jelinek
parent eaccea2afa
commit 5dcf6b7f1c
4 changed files with 26 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
gcc/ChangeLog
View File

@ -1,3 +1,9 @@
2012-06-26 Jakub Jelinek <jakub@redhat.com>
PR tree-optimization/53748
* tree-ssa-phiopt.c (conditional_replacement): Only optimize
if arg0/arg1 have integral or pointer types.
2012-06-25 Richard Henderson <rth@redhat.com>
* config/i386/sse.md (sse2_sse4_1): Remove code attr.

5
gcc/testsuite/ChangeLog
View File

@ -1,3 +1,8 @@
2012-06-26 Jakub Jelinek <jakub@redhat.com>
PR tree-optimization/53748
* gcc.c-torture/compile/pr53748.c: New test.
2012-06-25 Jason Merrill <jason@redhat.com>
PR c++/53498

9
gcc/testsuite/gcc.c-torture/compile/pr53748.c Normal file
View File

@ -0,0 +1,9 @@
/* PR tree-optimization/53748 */
typedef unsigned int V __attribute__ ((__vector_size__ (sizeof (int) * 4)));
void
foo (int x, V *y)
{
*y = x ? ((V) { ~0U, ~0U, ~0U, ~0U }) : ((V) { 0, 0, 0, 0 });
}

8
gcc/tree-ssa-phiopt.c
View File

@ -615,8 +615,12 @@ conditional_replacement (basic_block cond_bb, basic_block middle_bb,
bool neg;
/* FIXME: Gimplification of complex type is too hard for now. */
if (TREE_CODE (TREE_TYPE (arg0)) == COMPLEX_TYPE
|| TREE_CODE (TREE_TYPE (arg1)) == COMPLEX_TYPE)
/* We aren't prepared to handle vectors either (and it is a question
if it would be worthwhile anyway). */
if (!(INTEGRAL_TYPE_P (TREE_TYPE (arg0))
|| POINTER_TYPE_P (TREE_TYPE (arg0)))
|| !(INTEGRAL_TYPE_P (TREE_TYPE (arg1))
|| POINTER_TYPE_P (TREE_TYPE (arg1))))
return false;
/* The PHI arguments have the constants 0 and 1, or 0 and -1, then