diff options
| author | Chung-Lin Tang <cltang@codesourcery.com> | 2011-03-18 13:22:42 +0000 |
|---|---|---|
| committer | Chung-Lin Tang <cltang@gcc.gnu.org> | 2011-03-18 13:22:42 +0000 |
| commit | 2abbb0fe9e51ba2b1235f8cddf4953bfb8e48e93 (patch) | |
| tree | 18cdba4152f2ab36da07d5c34401ec0c9574f3f2 /gcc/combine.c | |
| parent | 12f9dfdaf15d081481be68101271b009ae2eea15 (diff) | |
| download | gcc-2abbb0fe9e51ba2b1235f8cddf4953bfb8e48e93.zip gcc-2abbb0fe9e51ba2b1235f8cddf4953bfb8e48e93.tar.gz gcc-2abbb0fe9e51ba2b1235f8cddf4953bfb8e48e93.tar.bz2 | |
combine.c (try_combine): Do simplification only call of subst() on i2 even when i1 is present.
2011-03-18 Chung-Lin Tang <cltang@codesourcery.com>
* combine.c (try_combine): Do simplification only call of
subst() on i2 even when i1 is present. Update comments.
testsuite/
* gcc.target/arm/unsigned-extend-1.c: New.
From-SVN: r171138
Diffstat (limited to 'gcc/combine.c')
| -rw-r--r-- | gcc/combine.c | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/gcc/combine.c b/gcc/combine.c index f9d33b3..fe3302a 100644 --- a/gcc/combine.c +++ b/gcc/combine.c @@ -3063,7 +3063,7 @@ try_combine (rtx i3, rtx i2, rtx i1, rtx i0, int *new_direct_jump_p) /* It is possible that the source of I2 or I1 may be performing an unneeded operation, such as a ZERO_EXTEND of something that is known to have the high part zero. Handle that case - by letting subst look at the innermost one of them. + by letting subst look at the inner insns. Another way to do this would be to have a function that tries to simplify a single insn instead of merging two or more @@ -3088,11 +3088,9 @@ try_combine (rtx i3, rtx i2, rtx i1, rtx i0, int *new_direct_jump_p) subst_low_luid = DF_INSN_LUID (i1); i1src = subst (i1src, pc_rtx, pc_rtx, 0, 0); } - else - { - subst_low_luid = DF_INSN_LUID (i2); - i2src = subst (i2src, pc_rtx, pc_rtx, 0, 0); - } + + subst_low_luid = DF_INSN_LUID (i2); + i2src = subst (i2src, pc_rtx, pc_rtx, 0, 0); } n_occurrences = 0; /* `subst' counts here */ |