diff options
| author | Richard Guenther <rguenther@suse.de> | 2011-01-20 10:33:15 +0000 |
|---|---|---|
| committer | Richard Biener <rguenth@gcc.gnu.org> | 2011-01-20 10:33:15 +0000 |
| commit | ddd268f2e96ce171c774df8707d4b32feb4961e7 (patch) | |
| tree | 8941373d666fd609a025decd10e84016c5d9462c /gcc/tree-ssa-copyrename.c | |
| parent | 525d13b031ea53d127623316d6c7751ca89cb591 (diff) | |
| download | gcc-ddd268f2e96ce171c774df8707d4b32feb4961e7.zip gcc-ddd268f2e96ce171c774df8707d4b32feb4961e7.tar.gz gcc-ddd268f2e96ce171c774df8707d4b32feb4961e7.tar.bz2 | |
re PR other/47167 (Performance regression in numerical code)
2011-01-20 Richard Guenther <rguenther@suse.de>
PR tree-optimization/47167
* tree-ssa-copyrename.c (copy_rename_partition_coalesce):
Revert previous change, only avoid enumeral type changes.
From-SVN: r169050
Diffstat (limited to 'gcc/tree-ssa-copyrename.c')
| -rw-r--r-- | gcc/tree-ssa-copyrename.c | 11 |
1 files changed, 8 insertions, 3 deletions
diff --git a/gcc/tree-ssa-copyrename.c b/gcc/tree-ssa-copyrename.c index abee6b9..9c51d11 100644 --- a/gcc/tree-ssa-copyrename.c +++ b/gcc/tree-ssa-copyrename.c @@ -226,11 +226,16 @@ copy_rename_partition_coalesce (var_map map, tree var1, tree var2, FILE *debug) ign2 = false; } - /* Don't coalesce if the two variables are not of the same type. */ - if (TREE_TYPE (root1) != TREE_TYPE (root2)) + /* Don't coalesce if the two variables aren't type compatible . */ + if (!types_compatible_p (TREE_TYPE (root1), TREE_TYPE (root2)) + /* There is a disconnect between the middle-end type-system and + VRP, avoid coalescing enum types with different bounds. */ + || ((TREE_CODE (TREE_TYPE (root1)) == ENUMERAL_TYPE + || TREE_CODE (TREE_TYPE (root2)) == ENUMERAL_TYPE) + && TREE_TYPE (root1) != TREE_TYPE (root2))) { if (debug) - fprintf (debug, " : Different types. No coalesce.\n"); + fprintf (debug, " : Incompatible types. No coalesce.\n"); return false; } |