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]

[PATCH PR62151]Fix REG_DEAD note distribution issue by using right ELIM_I0/ELIM_I1

From: "Bin Cheng" <bin dot cheng at arm dot com>
To: <gcc-patches at gcc dot gnu dot org>
Cc: "Jeff Law" <law at redhat dot com>, <segher at kernel dot crashing dot org>
Date: Thu, 11 Dec 2014 18:02:14 +0800
Subject: [PATCH PR62151]Fix REG_DEAD note distribution issue by using right ELIM_I0/ELIM_I1
Authentication-results: sourceware.org; auth=none

Hi,
As described both in the PR and patch comments, this patch fixes PR62151 by
setting right value to ELIM_I0/ELIM_I1 when distributing REG_DEAD notes from
i0/i1.  It is said that distribute_notes had caused many bugs in the past.
I think it still has bug in it, as noted in the PR.  This patch doesn't
touch distribute_notes because we are in stage3 and I want to have more
discussion on it.
Bootstrap and test on x86_64.  aarch64 is ongoing.  So is it ok?

2014-12-11  Bin Cheng  <bin.cheng@arm.com>

	PR rtl-optimization/62151
	* combine.c (try_combine): Reset elim_i0 and elim_i1 when
	distributing notes from i0notes or i1notes, this time don't
	check whether newi2pat sets i1dest or i0dest.

gcc/testsuite/ChangeLog
2014-12-11  Bin Cheng  <bin.cheng@arm.com>

	PR rtl-optimization/62151
	* gcc.c-torture/execute/pr62151.c: New test.

Attachment: pr62151-20141211.txt
Description: Text document

Follow-Ups:
- Re: [PATCH PR62151]Fix REG_DEAD note distribution issue by using right ELIM_I0/ELIM_I1
  - From: Eric Botcazou

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