PATCH: RFA reload.c patch

Jim Wilson wilson@specifixinc.com
Wed Nov 12 23:52:00 GMT 2003


On Wed, 2003-11-12 at 15:26, Eric Christopher wrote:
> +		      && (LEGITIMATE_CONSTANT_P (x)
> +			  || PREFERRED_RELOAD_CLASS (x, class) != NO_REGS))

> +			     && (LEGITIMATE_CONSTANT_P (x)
> +				 || PREFERRED_RELOAD_CLASS(x, class) != NO_REGS))))

Those should be && operators not || operators, as both conditions must
be true.  That also allows you to get rid of some parentheses now.

In the second test, the x should be XEXP (XEXP (x, 0), 1), i.e. the same
test used in CONSTANT_P.

Otherwise, yes, I believe this is the right fix.
-- 
Jim Wilson, GNU Tools Support, http://www.SpecifixInc.com



More information about the Gcc-patches mailing list