This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
RE: Binutils Port - Infineon xc16x family.
- From: "Shrirang Khishti" <shrirangk at KPITCummins dot com>
- To: "Shrirang Khishti" <shrirangk at KPITCummins dot com>, "Nick Clifton" <nickc at redhat dot com>
- Cc: <binutils at sourceware dot org>, "Shrinivas Atre" <ShrinivasA at KPITCummins dot com>, "Anil Paranjape" <AnilP1 at KPITCummins dot com>, "Shilin Shakti" <shilins at KPITCummins dot com>
- Date: Fri, 3 Mar 2006 18:48:42 +0530
- Subject: RE: Binutils Port - Infineon xc16x family.
Hi
I forgot to send you the change log.
Change log
2006-03-03 Shrirang Khisti <shrirangk@kpitcummins.com)
cpu/xc16x.opc : Changed the Parse functions to get rid
Of assembler errors.
Regards
Shrirang
-----Original Message-----
From: Shrirang Khishti
Sent: Friday, March 03, 2006 6:41 PM
To: 'Nick Clifton'
Cc: binutils@sourceware.org; Shrinivas Atre; Anil Paranjape; Shilin
Shakti
Subject: RE: Binutils Port - Infineon xc16x family.
Hi Nick
Please find the patch with this mail for xc16x.opc file.
Best Regards
Shrirang Khisti
-----Original Message-----
From: Nick Clifton [mailto:nickc@redhat.com]
Sent: Tuesday, February 28, 2006 5:45 PM
To: Shrirang Khishti
Cc: binutils@sourceware.org; Shrinivas Atre; Anil Paranjape; Shilin
Shakti
Subject: Re: Binutils Port - Infineon xc16x family.
Hi Shrirang,
> I analyzed the failures and found that GAS failures are because of
> changes made in xc16x.opc file for removing the warnings.
Oops - sorry!
> In functions parse_hash , parse_sof, parse_pof, parse_seg, parse_pag
> If we change following code (as you have changed)
> Code1: "If ( condition )
> {
> Str ++ ;
> Return NULL;
> }"
>
> To
> Code2: "
> If (condition)
> Str ++ ;
> Return NULL ";
>
> It removes the warnings which is expected but then it is giving
> assembler failures.
>
> After I revert the changes u suggested, test-cases are working fine.
> Any comments from u on this part?
This is fine. Please accept my apologies for corrupting your code.
Please could contribute a patch to fix up my mistake.
Cheers
Nick