AM30/AM33 SP handling

Alexandre Oliva aoliva@cygnus.com
Tue Apr 25 16:58:00 GMT 2000


On Apr 25, 2000, Jeffrey A Law <law@cygnus.com> wrote:

>   In message < orhfcpdi8s.fsf@zecarneiro.lsd.ic.unicamp.br >you write:
>> Index: gcc/ChangeLog
>> from  Alexandre Oliva  <aoliva@cygnus.com>
>> 
>> * config/mn10300/mn10300.h (REG_CLASS_FROM_LETTER): Return
>> EXTENDED_REGS only if TARGET_AM33.
>> * config/mn10300/mn10300.md (movsi, addsi): Do not allow
>> reload for SP.
> This is fine.  Though the ChangeLog entry for movsi/addsi is weird.

Indeed :-)

> 	* mn10300.md (movsi, addsi patterns): Avoid exposing SP_REGS to
> 	register allocation and reloading unless absolutely necessary.

> The ChangeLog entry is also missing the change you made to the addsi3
> expander.  You need to add an entry for that change.

Thanks, fixed, checking in.

-- 
Alexandre Oliva    Enjoy Guaraná, see http://www.ic.unicamp.br/~oliva/
Cygnus Solutions, a Red Hat company        aoliva@{redhat, cygnus}.com
Free Software Developer and Evangelist    CS PhD student at IC-Unicamp
oliva@{lsd.ic.unicamp.br, gnu.org}   Write to mailing lists, not to me



More information about the Gcc-patches mailing list