[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: address@hidden: Bug#247346: Patch for the bug]
From: |
Hakan Ardo |
Subject: |
Re: address@hidden: Bug#247346: Patch for the bug] |
Date: |
Wed, 5 Jan 2005 16:39:15 +0100 |
User-agent: |
Mutt/1.5.6+20040907i |
On Wed, Jan 05, 2005 at 01:21:45PM +0000, Nick Clifton wrote:
> Hi Hakan,
>
> >The attached patch (when dropped into a debian/patches folder for
> >binutils-avr) provides a new flag to avr-as "--allow-dollars", which
> >overrides the default settings to always allow dollars in function
> >names. Behaviour without this flag is unchanged from standard avr-as.
>
> It seems to me that this is the wrong way to fix this problem. It is
> creating a new generic command line switch for GAS which is used to fix
> a problem with the AVR port. I would be much happier with a patch that
> only modifies AVR specific files (ie gas/config/tc-avr.c). A generic
> patch seems to me like unnecessary feature bloat.
NesC is not only supported on the avr, but on 8051 and TI MSP430 aswell and
possible even more arches in the future.
--
Hakan Ardo