Committed, Fix part of PR target/19065; an autoincrement-free mem constraint for CRIS.

Mark Mitchell mark@codesourcery.com
Mon Feb 28 14:13:00 GMT 2005


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



More information about the Gcc-patches mailing list