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: Committed, Fix part of PR target/19065; an autoincrement-freemem constraint for CRIS.

From: Mark Mitchell <mark at codesourcery dot com>
To: Hans-Peter Nilsson <hans-peter dot nilsson at axis dot com>
Cc: gcc-patches at gcc dot gnu dot org
Date: Sun, 27 Feb 2005 21:11:30 -0800
Subject: Re: Committed, Fix part of PR target/19065; an autoincrement-freemem constraint for CRIS.
Organization: CodeSourcery, LLC
References: <200502280224.j1S2ONMG016791@ignucius.se.axis.com>

Hans-Peter Nilsson wrote:

This is necessary for PR target/19065.  Actually, 'Q' was used
all along in cris.md to only mean (mem reg) and never (reg), so
the work there was mostly formatting adjustment and review to
make sure, which found the miss in "cmphi".

I'm content for backend maintainers to use their best judgement, so long as they act in keeping with the idea behind the regression-only rule; namely, that only conservative patches that fix very important problems are applied. Regressions are used as a proxy for that because we assume that people were using the previous compiler with whatever bugs it had. However, if there's a bug that's so severe that you think it's vital it be fixed, and the fix is entirely within target-dependent code, I'm happy for you to use your best judgement.

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

References:
- Committed, Fix part of PR target/19065; an autoincrement-free mem constraint for CRIS.
  - From: Hans-Peter Nilsson

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