Cyril Holweck wrote:

the patch for half of the newest CPU with 4 digits has a conflict.

simple compilation leads to the following:

msp430-ld -m msp430x2274 [...]
msp430-ld: cannot represent machine `msp:220'



http://mspgcc.cvs.sourceforge.net/mspgcc/gcc/gcc-3.3/gcc/config/msp430/msp430.c?revision=1.90&view=markup

 MSP430_ISA_200 = 200,
 MSP430_ISA_210 = 210,
 MSP430_ISA_220 = 220,
 {"msp430x2274",  MSP430_ISA_220, 0},


conflicts with msp430-ld:

#define MSP430_ISA_20  20
#define MSP430_ISA_21  21
#define MSP430_ISA_22  22
 {"msp430x2274",  MSP430_ISA_22,  bfd_mach_msp22},


the latter should read instead

#define MSP430_ISA_200  200
#define MSP430_ISA_210  210
#define MSP430_ISA_220  220
 {"msp430x2274",  MSP430_ISA_220,  bfd_mach_msp22},


or better, but with a bit more patches to do

 {"msp430x2274",  MSP430_ISA_220,  bfd_mach_msp220},
That should be sorted out in CVS. I use 20, 21, and 22 consistently now.

Steve


Reply via email to