[patch,avr] device specific instruction support for avr devices

S, Pitchumani Pitchumani.S@atmel.com
Thu Mar 27 05:07:00 GMT 2014


> -----Original Message-----
> From: Denis Chertykov [mailto:chertykov@gmail.com]
> Sent: Tuesday, March 25, 2014 10:37 PM
> To: S, Pitchumani
> Cc: binutils@sourceware.org
> Subject: Re: [patch,avr] device specific instruction support for avr
> devices
> 
> 2014-03-24 12:23 GMT+04:00 S, Pitchumani <Pitchumani.S@atmel.com>:
> >> -----Original Message-----
> >> From: binutils-owner@sourceware.org [mailto:binutils-
> owner@sourceware.org]
> >> On Behalf Of S, Pitchumani
> >> Sent: Monday, March 03, 2014 11:21 AM
> >> To: binutils@sourceware.org
> >> Cc: chertykov@gmail.com
> >> Subject: [patch,avr] device specific instruction support for avr
> devices
> >>
> >> Hi,
> >>
> >> Few AVR Xmega devices have specific instruction support than the
> >> architecture
> >> it belongs to. For example atxmega128b1 device has RMW instructions
> >> (XCH,LAC,
> >> LAS and LAT) support, but not all avrxmega6 devices have.
> >>
> >> Now, avr-gcc passes architecture name to assembler instead of device
> name.
> >> So,
> >> RMW instructions are not recognized (illegal opcode error) by
> assembler.
> >>
> >> I have attached a patch to address this issue in assembler.
> >> - It adds a option '-mrmw' to assembler.
> >> - Based on -mrmw option device specific instructions are added to
> current
> >>   instruction set (mcu_types[i].isa | AVR_ISA_RMW)
> >> Now assembler can recognize rmw instructions for respective devices.
> >>
> >> It is necessary to update gcc to pass -mrmw option to assembler if the
> >> selected device has rmw instructions. I'll send a patch for gcc as
> well.
> >>
> >> Please review the attached patch and give your suggestions.
> >
> > Hi,
> >
> > I have updated the patch to include -mrmw option to binutils.
> 
> Where is the patch ?
> 
> Denis.


Sorry, now I have attached the patch.

Regards,
Pitchumani

gas/ChangeLog
2014-03-27 Pitchumani Sivanupandi <pitchumani.s@atmel.com>
 
    * config/tc-avr.c: Add specified_mcu variable for selected mcu.
    (enum options): add OPTION_ISA_RMW for -mrmw option.
    (struct option md_longopts): Add mrmw option.
    (md_show_usage): add -mrmw option description.
    (md_parse_option): Update isa details if -mrmw option specified.
    * doc/c-avr.texi: Add doc for new option -mrmw.
 
gas/testsuite/ChangeLog
2014-02-27 Pitchumani Sivanupandi <pitchumani.s@atmel.com>
 
    * gas/avr/avr.exp: Run new tests for avr target.
    * gas/avr/rmw.d: Add test for additional ISA support.
    * gas/avr/rmw.s: Ditto.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: dev-specific-feature-avr-as.patch
Type: application/octet-stream
Size: 5273 bytes
Desc: dev-specific-feature-avr-as.patch
URL: <https://sourceware.org/pipermail/binutils/attachments/20140327/93965edf/attachment.obj>


More information about the Binutils mailing list