This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
{Patch] }Treat avr as elf target in nm.exp
- From: Senthil Kumar Selvaraj <senthil_kumar dot selvaraj at atmel dot com>
- To: Binutils <binutils at sourceware dot org>
- Cc: Denis Chertykov <chertykov at gmail dot com>, Nick Clifton <nickc at redhat dot com>
- Date: Sat, 23 Apr 2016 23:24:19 +0530
- Subject: {Patch] }Treat avr as elf target in nm.exp
- Authentication-results: sourceware.org; auth=none
Hi,
This patch adds avr to the list of ELF targets in nm.exp - without
this, the nm --size-sort test fails for the AVR target, as it uses
nm-1.s as the source file, instead of nm-elf-1.s (which has the .size
directive to set symbol size).
I tried adding avr-unknown-none to is_elf_format itself, but that
causes a lot of failures in gas and LD, just as the comment says :)
I intend to follow up on those failures as well, but until then, this
patch lets the specific testcase pass.
Regards
Senthil
binutils/testsuite/ChangeLog
2016-04-23 Senthil Kumar Selvaraj <senthil_kumar.selvaraj@atmel.com>
* binutils-all/nm.exp: Add avr to elf
target list.
diff --git a/binutils/testsuite/binutils-all/nm.exp b/binutils/testsuite/binutils-all/nm.exp
index 94f7f9c..34454f8 100644
--- a/binutils/testsuite/binutils-all/nm.exp
+++ b/binutils/testsuite/binutils-all/nm.exp
@@ -163,6 +163,7 @@ if [regexp $want $got] then {
# all the ELF variants, but adding the patterns here to that proc then
# introduces a whole slew of new regressions in the GAS and LD testsuites.
if { [is_elf_format]
+ || [istarget avr-unknown-none]
|| [istarget *-*-dragonfly*]
|| [istarget *-*-*elf]
|| [istarget *-*-freebsd*]