aboutsummaryrefslogtreecommitdiff
path: root/gcc/tree-ssa-loop-ch.c
diff options
context:
space:
mode:
authorAlexandre Oliva <aoliva@redhat.com>2018-11-09 10:16:09 +0000
committerAlexandre Oliva <aoliva@gcc.gnu.org>2018-11-09 10:16:09 +0000
commitd4920f402af2591fa9864863f8c4ddcad1f9ebc1 (patch)
treefca3caf0766d3151c362068508b4da0f0763b702 /gcc/tree-ssa-loop-ch.c
parentb759ea284782b95cf0196c8e813e3ab7e1233c47 (diff)
downloadgcc-d4920f402af2591fa9864863f8c4ddcad1f9ebc1.zip
gcc-d4920f402af2591fa9864863f8c4ddcad1f9ebc1.tar.gz
gcc-d4920f402af2591fa9864863f8c4ddcad1f9ebc1.tar.bz2
[PR86438] compare-elim: cope with set of in_b
When in_a resolves to a register set in the prev_clobber insn, we may use the SET_SRC for the compare instead. However, when in_b so resolves, we proceed to use the reg with its earlier value. When both resolve to the same register and prev_clobber is an insn that modifies the register, this arrangement may cause the compare to match (when it shouldn't) and the elimination of the compare to incorrectly succeed. (set (reg 1) (plus (reg 1) (const_int N))) (set (reg 2) (reg 1)) (set (reg flags) (compare (reg 1) (reg 2))) in_a: (reg 1) --> (plus (reg 1) (const_int N)) in_b: (reg 2) -> (reg 1) -/> oops (parallel [ (set (reg flags) (compare (plus (reg 1) (const_int N)) (reg 1))) ;; should be (plus...) (set (reg 1) (plus (reg 1) (const_int N)))]) (set (reg 2) (reg 1)) This patch arranges for in_b to also undergo SET_SRC substitution when appropriate, with a shortcut for when in_a and in_b are the same rtx. for gcc/ChangeLog PR rtl-optimization/86438 * compare-elim.c (try_eliminate_compare): Use SET_SRC instead of in_b for the compare if in_b is SET_DEST. for gcc/testsuite/ChangeLog PR rtl-optimization/86438 * gcc.dg/torture/pr86438.c: New. From-SVN: r265957
Diffstat (limited to 'gcc/tree-ssa-loop-ch.c')
0 files changed, 0 insertions, 0 deletions