This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH, 4.8] Backport "Fix debug-insn sensitivity in RA" patch to 4.8



On 2015-02-18 12:25 PM, Uros Bizjak wrote:
Hello!

Richard's patch at [1] is needed to fix a bootstrap failure on
alpha-linux-gnu on 4.8 branch. Without the patch, IRA creates
different sequences, depending on the presence of -g option.

One of the many comparison failures is in expr.c, where
-fdebug-compare fails with:

...

2015-02-18  Uros Bizjak  <ubizjak@gmail.com>

     Backport from mainline
     2013-09-08  Richard Sandiford  <rdsandiford@googlemail.com>

     * ira.c (update_equiv_regs): Only call set_paradoxical_subreg
     for non-debug insns.
     * lra.c (new_insn_reg): Take the containing insn as a parameter.
     Only modify lra_reg_info[].biggest_mode if it's non-debug insn.
     (collect_non_operand_hard_regs, add_regs_to_insn_regno_info): Update
     accordingly.

testsuite/ChangeLog:

2015-02-18  Uros Bizjak  <ubizjak@gmail.com>

     Backport from mainline
     2013-09-08  Richard Sandiford  <rdsandiford@googlemail.com>

     * g++.dg/debug/ra1.C: New test.

Patch was bootstrapped and regression tested on 86_64-linux-gnu
{,-m32} and alpha-linux-gnu, where it fixes all bootstrap comparison
failures.

The results are at [3], gfortran failures will be fixed by alias.c
backport, after the bootstrap problem is cured.

OK for branch?


Yes, Uros.  Thanks of working on this.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]