This is the mail archive of the
mailing list for the GCC project.
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.