This is the mail archive of the binutils@sourceware.org mailing list for the binutils project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [patch, avr, size] add --mcu and -C options from winavr

From: Alan Modra <amodra at gmail dot com>
To: Jonathan Kotta <jpkotta at gmail dot com>
Cc: binutils at sourceware dot org
Date: Thu, 8 Nov 2012 11:36:13 +1030
Subject: Re: [patch, avr, size] add --mcu and -C options from winavr
References: <CALDneatWM6zkqcDbYKmdxrd3DOeZvsC1O4KyYKPGz07SLZGtQQ@mail.gmail.com>

On Wed, Nov 07, 2012 at 05:46:47PM -0600, Jonathan Kotta wrote:
> This patch is applied by several Linux distributions, and shows you
> how much flash and RAM a program uses as a percentage of the total for
> a given chip.  It would be nice if it was upstream.  It should only be
> applied when building for AVR, but I'm unsure of how to do that, so I
> would appreciate help with that.

I think this would be better as a separate utility.
- Your added code increases size.c by almost 50%.
- Your code doesn't use anything in size.c besides option handling code.
- Your code doesn't support an existing option, --radix.

-- 
Alan Modra
Australia Development Lab, IBM

References:
- [patch, avr, size] add --mcu and -C options from winavr
  - From: Jonathan Kotta

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]