This is the mail archive of the gcc@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: Proposal to postpone release of 5.2 for a week [Was: Re: patch to fix PR66782]




On 07/10/2015 04:09 AM, Richard Biener wrote:
On Thu, 9 Jul 2015, Uros Bizjak wrote:

Hello!

The patch was bootstrapped and tested on x86/x86-64.

Committed as rev. 225618.

2015-07-09  Vladimir Makarov  <vmakarov@redhat.com>

         PR rtl-optimization/66782
         * lra-int.h (struct lra_insn_recog_data): Add comment about
         clobbered hard regs for arg_hard_regs.
         * lra.c (lra_set_insn_recog_data): Add clobbered hard regs.
         * lra-lives.c (process_bb_lives): Process clobbered hard regs.
         Add condition for processing used hard regs.
         * lra-constraints.c (update_ebb_live_info, inherit_in_ebb):
         Process clobbered hard regs.
I would like to nominate this patch for gcc-5.2 release. According to
downstream bugreport [1], gcc-5.1 is unusable for 64-bit wine:

"Breaks all of wine, no easy workaround -> blocker."

Due to severity of this bug, and importance of Wine, I'd like to
postpone the 5.2 release for a week, so this bug gets some testing in
the mainline, before it is backported to gcc-5 branch

[1] https://bugs.winehq.org/show_bug.cgi?id=38653
Hm.  I'd rather burn this with a RC2 released soon or defer it to
GCC 5.3.  The patch looks kind-of straight-forward, likely not
affecting anything else (to my naiive eyes...).

So - please get it committed to the GCC 5 branch as soon as possible.
A GCC 5.2 RC2 will be done on Monday latest then (possibly during the
weekend if I find the time to do it).

Note this opens the window for other important wrong-code fixes - please
CC me on any you'd like to propose for GCC 5.2 and wait for my
explicit approval.


Thanks, Richard. I believe the patch is safe. So I've backported the patch to the branch as rev. 225674. I successfully bootstrapped and tested it on x86-64.


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