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

re PR rtl-optimization/80385 (Segfault in commutative_operand_precedence() rtlanal.c:3373)

Browse Source 
PR rtl-optimization/80385
	* simplify-rtx.c (simplify_unary_operation_1): Don't transform
	(not (neg X)) into (plus X -1) for complex or non-integral modes.

	* g++.dg/opt/pr80385.C: New test.

From-SVN: r246850
This commit is contained in:
Jakub Jelinek 2017-04-11 19:19:56 +02:00 committed by Jakub Jelinek
parent fbc698e0f4
commit 8585103f05
4 changed files with 27 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
gcc/ChangeLog
View File

@ -1,5 +1,9 @@
2017-04-11 Jakub Jelinek <jakub@redhat.com>
PR rtl-optimization/80385
* simplify-rtx.c (simplify_unary_operation_1): Don't transform
(not (neg X)) into (plus X -1) for complex or non-integral modes.
PR libgomp/80394
* omp-low.c (scan_omp_task): Don't optimize away empty tasks
if they have any depend clauses.

6
gcc/simplify-rtx.c
View File

@ -932,8 +932,10 @@ simplify_unary_operation_1 (enum rtx_code code, machine_mode mode, rtx op)
&& XEXP (op, 1) == constm1_rtx)
return simplify_gen_unary (NEG, mode, XEXP (op, 0), mode);
/* Similarly, (not (neg X)) is (plus X -1). */
if (GET_CODE (op) == NEG)
/* Similarly, (not (neg X)) is (plus X -1). Only do this for
modes that have CONSTM1_RTX, i.e. MODE_INT, MODE_PARTIAL_INT
and MODE_VECTOR_INT. */
if (GET_CODE (op) == NEG && CONSTM1_RTX (mode))
return simplify_gen_binary (PLUS, mode, XEXP (op, 0),
CONSTM1_RTX (mode));

5
gcc/testsuite/ChangeLog
View File

@ -1,3 +1,8 @@
2017-04-11 Jakub Jelinek <jakub@redhat.com>
PR rtl-optimization/80385
* g++.dg/opt/pr80385.C: New test.
2017-04-11 Martin Liska <mliska@suse.cz>
PR ipa/80212

14
gcc/testsuite/g++.dg/opt/pr80385.C Normal file
View File

@ -0,0 +1,14 @@
// PR rtl-optimization/80385
// { dg-do compile { target { i?86-*-* x86_64-*-* } } }
// { dg-options "-Ofast -msse2" }
#include <x86intrin.h>
__m128 a, e;
struct A { __m128 b; A (); A (__m128 x) : b(x) {} };
A operator+ (A, A);
A operator- (A) { __m128 c = -a; return c; }
A foo (A x) { __m128 d = x.b; return _mm_andnot_ps (d, e); }
struct B { A n[1]; };
void bar (B x) { A f = foo (x.n[0]); A g = f + A (); }
void baz () { B h; B i; A j; i.n[0] = -j; h = i; B k = h; bar (k); }