[PATCH 3/5] New target port: Andes 'nds32'. (gas)

Kuan-Lin Chen kuanlinchentw@gmail.com
Wed Dec 11 08:13:00 GMT 2013


Sorry for forgetting generating diff-patch form gas/NEWS.
Therefore, I send a new patch again.

Please fine the attached file.

2013/12/11 Kuan-Lin Chen <kuanlinchentw@gmail.com>:
> Hi,  Nick
>
> Thanks for your review and comment.
>
> I have updated this patch as follows:
> 1. Add nds32 specification into document.
> 2. Add testsuite gas/testsuite/gas/nds32.
> 3. Announce nds32 support.
> 4. Replace all PTR uses.
>
> Please find the attached file.
>
> 2013/12/6 nick clifton <nickc@redhat.com>:
>> Hi Kuan-Lin,
>>
>>   There are a few problems with this patch:
>>
>>   1.  There is no addition to the assembler documentation.
>>
>>       I would like to see a gas/doc/c-nds32.texi file which includes
>> documentation for the NDS32 specific command line options, pseudo-ops
>> and synthetic instructions.
>>
>>
>>   2.  There is no testsuite.
>>
>>       There ought to be a gas/testsuite/gas/nds32 directory containing at
>> least one test, (but preferably a lot more), that checks the assembly of
>> NDS32 instructions.
>>
>>
>>   3.  There is no NEWS announcement.
>>
>>       The file gas/NEWS should be patched to include a sentance announcing
>> the additon of support for the NDS32.
>>
>>
>>   4.  The use of the PTR macro is deprecated.
>>
>>       Please replace all uses of the PTR macro with "void *".
>>
>>
>> Cheers
>>   Nick
>>
>
>
>
> --
> Best regards,
> Kuan-Lin Chen.
> kuanlinchentw@gmail.com



-- 
Best regards,
Kuan-Lin Chen.
kuanlinchentw@gmail.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: binutils-3-gas.patch
Type: application/octet-stream
Size: 209023 bytes
Desc: not available
URL: <https://sourceware.org/pipermail/binutils/attachments/20131211/072c8fa7/attachment.obj>


More information about the Binutils mailing list