Re: [Patch,AVR]: Clean up SFR offset usage: %i for CONST_INT

2011-11-25 Thread Denis Chertykov
2011/11/25 Georg-Johann Lay > > Georg-Johann Lay wrote: > > Denis Chertykov wrote: > >> 2011/11/20 Georg-Johann Lay <.>: > >>> Subtracting 0x20 to get the SFR address from a RAM address is scattered > >>> all > >>> over the backend.  The patch makes - PRINT_OPERAND_PUNCT_VALID_P and uses > >

Re: [Patch,AVR]: Clean up SFR offset usage: %i for CONST_INT

2011-11-25 Thread Georg-Johann Lay
Georg-Johann Lay wrote: > Denis Chertykov wrote: >> 2011/11/20 Georg-Johann Lay <.>: >>> Subtracting 0x20 to get the SFR address from a RAM address is scattered all >>> over the backend. The patch makes - PRINT_OPERAND_PUNCT_VALID_P and uses >>> %- to >>> subtract the SFR offset instead of ha

Re: [Patch,AVR]: Clean up SFR offset usage, take #2

2011-11-20 Thread Denis Chertykov
2011/11/20 Georg-Johann Lay : > Denis Chertykov wrote: >> 2011/11/20 Georg-Johann Lay <.>: >>> Subtracting 0x20 to get the SFR address from a RAM address is scattered all >>> over the backend.  The patch makes - PRINT_OPERAND_PUNCT_VALID_P and uses >>> %- to >>> subtract the SFR offset instead

Re: [Patch,AVR]: Clean up SFR offset usage, take #2

2011-11-20 Thread Georg-Johann Lay
Denis Chertykov wrote: > 2011/11/20 Georg-Johann Lay <.>: >> Subtracting 0x20 to get the SFR address from a RAM address is scattered all >> over the backend. The patch makes - PRINT_OPERAND_PUNCT_VALID_P and uses %- >> to >> subtract the SFR offset instead of hard coded magic number 0x20 all

Re: [Patch,AVR]: Clean up SFR offset usage.

2011-11-19 Thread Denis Chertykov
2011/11/20 Georg-Johann Lay : > Subtracting 0x20 to get the SFR address from a RAM address is scattered all > over the backend.  The patch makes - PRINT_OPERAND_PUNCT_VALID_P and uses %- > to > subtract the SFR offset instead of hard coded magic number 0x20 all over the > place.  The offset is sto

[Patch,AVR]: Clean up SFR offset usage.

2011-11-19 Thread Georg-Johann Lay
Subtracting 0x20 to get the SFR address from a RAM address is scattered all over the backend. The patch makes - PRINT_OPERAND_PUNCT_VALID_P and uses %- to subtract the SFR offset instead of hard coded magic number 0x20 all over the place. The offset is stored in a new field base_arch_s.sfr_offset