procyberian/gcc
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix violations of self-assignment check in GCC source.

Browse source 
From-SVN: r162492
This commit is contained in:
Le-Chun Wu 2010-07-23 22:20:45 +00:00 • committed by Le-Chun Wu
parent df161b697c
commit 3d07b7cde4
5 changed files with 18 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
gcc/ChangeLog
View file

@ -1,3 +1,13 @@
2010-07-23 Le-Chun Wu <lcwu@google.com>
* omega.c (omega_eliminate_redundant): Remove a self-assign statement.
* tree-ssa-ccp.c (ccp_lattice_meet): Remove a self-assign statement
and an unnecessary assignment.
* dbxout.c (DEBUGGER_ARG_OFFSET): Change OFFSET to OFFSET+0 to avoid
self-assign warning.
* config/i386/i386.c (ix86_vectorize_builtin_vec_perm): Remove
unnecessary self-init.
2010-07-23 Richard Guenther <rguenther@suse.de>
PR lto/43071

2
gcc/config/i386/i386.c
View file

@ -30012,7 +30012,7 @@ ix86_vectorize_builtin_vec_perm (tree vec_type, tree *mask_type)
tree itype = TREE_TYPE (vec_type);
bool u = TYPE_UNSIGNED (itype);
enum machine_mode vmode = TYPE_MODE (vec_type);
enum ix86_builtins fcode = fcode; /* Silence bogus warning. */
enum ix86_builtins fcode;
bool ok = TARGET_SSE2;
switch (vmode)

7
gcc/dbxout.c
View file

@ -289,9 +289,12 @@ static const char *base_input_file;
#endif
/* A C expression for the integer offset value of an argument (N_PSYM)
having address X (an RTX). The nominal offset is OFFSET. */
having address X (an RTX). The nominal offset is OFFSET.
Note that we use OFFSET + 0 here to avoid the self-assign warning
when the macro is called in a context like
number = DEBUGGER_ARG_OFFSET(number, X) */
#ifndef DEBUGGER_ARG_OFFSET
#define DEBUGGER_ARG_OFFSET(OFFSET, X) (OFFSET)
#define DEBUGGER_ARG_OFFSET(OFFSET, X) (OFFSET + 0)
#endif
/* This obstack holds the stab string currently being constructed. We

1
gcc/omega.c
View file

@ -2213,7 +2213,6 @@ omega_eliminate_redundant (omega_pb pb, bool expensive)
|| pb->geqs[e3].color == omega_red)
goto nextE3;
alpha3 = alpha3;
/* verify alpha1*v1+alpha2*v2 = alpha3*v3 */
for (k = pb->num_vars; k >= 1; k--)
if (alpha3 * pb->geqs[e3].coef[k]

5
gcc/tree-ssa-ccp.c
View file

@ -730,9 +730,8 @@ ccp_lattice_meet (prop_value_t *val1, prop_value_t *val2)
Ci M Cj = VARYING if (i != j)
If these two values come from memory stores, make sure that
they come from the same memory reference. */
val1->lattice_val = CONSTANT;
val1->value = val1->value;
they come from the same memory reference.
Nothing to do. VAL1 already contains the value we want. */
}
else
{