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: [4.0] Approved PR21254 fix different from patch applied to mainline?

From: Mark Mitchell <mark at codesourcery dot com>
To: Paolo Bonzini <paolo dot bonzini at lu dot unisi dot ch>
Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>, Zdenek Dvorak <rakdver at atrey dot karlin dot mff dot cuni dot cz>
Date: Thu, 18 Aug 2005 10:12:48 +0100
Subject: Re: [4.0] Approved PR21254 fix different from patch applied to mainline?
References: <43034CCD.4000102@lu.unisi.ch>

Paolo Bonzini wrote:

The PR/21254 has a fix for the 4.0 branch, approved by Mark, that I applied a couple of hours ago. However the patch is different from what Zdenek committed to mainline. It lacks the following line:
Index: loop-iv.c
===================================================================
RCS file: /cvs/gcc/gcc/gcc/loop-iv.c,v
retrieving revision 2.26.12.1
diff -p -u -b -r2.26.12.1 loop-iv.c
--- loop-iv.c   17 Aug 2005 12:45:01 -0000      2.26.12.1
+++ loop-iv.c   17 Aug 2005 14:35:46 -0000
@@ -2551,6 +2551,7 @@ zero_iter:
  desc->const_iter = true;
  desc->niter = 0;
  desc->niter_max = 0;
+  desc->noloop_assumptions = NULL_RTX;

I think that, absent any obvious reason for the discrepancy, we should match mainline. The patch is fine by me. If Zdenek thinks it's actively wrong, he'll let you know.

Thanks,

--
Mark Mitchell
CodeSourcery, LLC
mark@codesourcery.com
(916) 791-8304

References:
- [4.0] Approved PR21254 fix different from patch applied to mainline?
  - From: Paolo Bonzini

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