From c8b90a13c729f6fb33a97c4f4cd8116e5ef7fdf1 Mon Sep 17 00:00:00 2001 From: Segher Boessenkool Date: Thu, 22 Jan 2015 01:35:44 +0100 Subject: [PATCH] re PR rtl-optimization/64682 (wrong code at -O2 and -O3 on x86_64-linux-gnu) PR rtl-optimization/64682 * combine.c (distribute_notes): When moving a death note for a register that is set in the new I2, make sure to put it before that new I2. PR rtl-optimization/64682 * gcc.c-torture/execute/pr64682.c: New file. From-SVN: r219981 --- gcc/ChangeLog | 7 +++++++ gcc/combine.c | 5 +++++ gcc/testsuite/ChangeLog | 5 +++++ 3 files changed, 17 insertions(+) diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 756c37d..c292bfc 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,10 @@ +2015-01-21 Segher Boessenkool + + PR rtl-optimization/64682 + * combine.c (distribute_notes): When moving a death note for + a register that is set in the new I2, make sure to put it + before that new I2. + 2015-01-21 David Edelsohn * config/rs6000/rs6000.c (rs6000_file_start): Use rs6000_isa_flags diff --git a/gcc/combine.c b/gcc/combine.c index 597aa80..5c763b4 100644 --- a/gcc/combine.c +++ b/gcc/combine.c @@ -13684,6 +13684,11 @@ distribute_notes (rtx notes, rtx_insn *from_insn, rtx_insn *i3, rtx_insn *i2, || rtx_equal_p (XEXP (note, 0), elim_i0)) break; tem_insn = i3; + /* If the new I2 sets the same register that is marked dead + in the note, the note now should not be put on I2, as the + note refers to a previous incarnation of the reg. */ + if (i2 != 0 && reg_set_p (XEXP (note, 0), PATTERN (i2))) + tem_insn = i2; } if (place == 0) diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index f093c78..2a54aff 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2015-01-21 Segher Boessenkool + + PR rtl-optimization/64682 + * gcc.c-torture/execute/pr64682.c: New file. + 2015-01-21 Jakub Jelinek PR debug/64511 -- 2.7.4