Fix some AVR test failures.
authorSenthil Kumar Selvaraj <senthil_kumar.selvaraj@atmel.com>
Wed, 4 May 2016 12:03:52 +0000 (13:03 +0100)
committerNick Clifton <nickc@redhat.com>
Wed, 4 May 2016 12:03:52 +0000 (13:03 +0100)
binutils* testsuite/lib/binutils-common.exp (is_elf_format): Add avr-*-*.

ld * testsuite/ld-elf/pr18735.d: Allow other symbols.
* testsuite/ld-elf/sec64k.exp: Skip 64ksec for avr.
* testsuite/ld-gc/pr14265.d: Allow other symbols.
* testsuite/ld-plugin/plugin.exp: Add PR ld/17973 to
plugin_tests only if check_shared_lib_support is true.
* testsuite/ld-selective/selective.exp: Add --section-start
flag for avr.

binutils/ChangeLog
binutils/testsuite/lib/binutils-common.exp
ld/ChangeLog
ld/testsuite/ld-elf/pr18735.d
ld/testsuite/ld-elf/sec64k.exp
ld/testsuite/ld-gc/pr14265.d
ld/testsuite/ld-plugin/plugin.exp
ld/testsuite/ld-selective/selective.exp

index d2f6e95693f3eea4e77abadfcd38475e1aa7aeae..12e8bb2dac4a9f039682608f1a6da9af6c437937 100644 (file)
@@ -1,3 +1,7 @@
+2016-05-04  Senthil Kumar Selvaraj  <senthil_kumar.selvaraj@atmel.com>
+
+       * testsuite/lib/binutils-common.exp (is_elf_format): Add avr-*-*.
+
 2016-05-03  Nick Clifton  <nickc@redhat.com>
 
        * po/sv.po: Updated Swedish translation.
index dbbbc792243235b39bd544db7a4c59499f1cf885..5f170127bb98fe273cb32723aa099e9de83e9d2b 100644 (file)
@@ -27,44 +27,42 @@ proc is_elf_format {} {
        return 1;
     }
 
-    if { ![istarget *-*-sysv4*]
-        && ![istarget *-*-unixware*]
+    if {    ![istarget *-*-eabi*]
         && ![istarget *-*-elf*]
-        && ![istarget *-*-eabi*]
-        && ![istarget *-*-rtems*]
-        && ![istarget hppa*64*-*-hpux*]
-        && ![istarget ia64-*-hpux*]
-        && ![istarget *-*-linux*]
+        && ![istarget *-*-freebsd*]
         && ![istarget *-*-gnu*]
-        && ![istarget *-*-nacl*]
-        && ![istarget frv-*-uclinux*]
-        && ![istarget bfin-*-uclinux]
-        && ![istarget sh*-*-uclinux*]
-        && ![istarget tic6x*-*-uclinux*]
         && ![istarget *-*-irix5*]
         && ![istarget *-*-irix6*]
-        && ![istarget *-*-freebsd*]
+        && ![istarget *-*-linux*]
+        && ![istarget *-*-nacl*]
         && ![istarget *-*-netbsd*]
         && ![istarget *-*-openbsd*]
-        && ![istarget *-*-solaris2*] } {
+        && ![istarget *-*-rtems*]
+        && ![istarget *-*-solaris2*]
+        && ![istarget *-*-sysv4*]
+        && ![istarget *-*-unixware*]
+        && ![istarget avr-*-*]
+        && ![istarget bfin-*-uclinux]
+        && ![istarget frv-*-uclinux*]
+        && ![istarget hppa*64*-*-hpux*]
+        && ![istarget ia64-*-hpux*]
+        && ![istarget sh*-*-uclinux*]
+        && ![istarget tic6x*-*-uclinux*] } {
        return 0
     }
 
-    if { [istarget i?86-*-freebsd\[12\].*] } {
-        return 0
-    }
-
-    if { [istarget *-*-linux*aout*]
+    if {    [istarget *-*-linux*aout*]
         || [istarget *-*-linux*ecoff*]
         || [istarget *-*-linux*oldld*]
+        || [istarget *-*-rtemscoff*]
         || [istarget h8500-*-rtems*]
-        || [istarget i960-*-rtems*]
-        || [istarget *-*-rtemscoff*] } {
+        || [istarget i?86-*-freebsd\[12\].*]
+        || [istarget i960-*-rtems*] } {
        return 0
     }
 
     if { ![istarget *-*-netbsdelf*]
-        && ([istarget *-*-netbsd*aout*]
+        && (   [istarget *-*-netbsd*aout*]
             || [istarget *-*-netbsdpe*]
             || [istarget arm*-*-netbsd*]
             || [istarget sparc-*-netbsd*]
@@ -75,7 +73,7 @@ proc is_elf_format {} {
        return 0
     }
 
-    if { [istarget arm-*-openbsd*]
+    if {    [istarget arm-*-openbsd*]
         || [istarget i386-*-openbsd\[0-2\].*]
         || [istarget i386-*-openbsd3.\[0-2\]]
         || [istarget m68*-*-openbsd*]
index 092d2e84671d2eecc1bde45d4904f4b55419b74d..8549baab7bb859179117191a4db45e68c1fe3427 100644 (file)
@@ -1,3 +1,13 @@
+2016-05-04  Senthil Kumar Selvaraj  <senthil_kumar.selvaraj@atmel.com>
+
+       * testsuite/ld-elf/pr18735.d: Allow other symbols.
+       * testsuite/ld-elf/sec64k.exp: Skip 64ksec for avr.
+       * testsuite/ld-gc/pr14265.d: Allow other symbols.
+       * testsuite/ld-plugin/plugin.exp: Add PR ld/17973 to
+       plugin_tests only if check_shared_lib_support is true.
+       * testsuite/ld-selective/selective.exp: Add --section-start
+       flag for avr.
+
 2016-05-03  Maciej W. Rozycki  <macro@imgtec.com>
 
        PR 10549
index bc19a827b6c481ea2073757e3a0d7e21003d45e0..5f64818dd9f82284bc5e5f28eb7ff9de97079aa8 100644 (file)
@@ -6,6 +6,8 @@ Symbol table '.symtab' contains .* entries:
 [      ]*[0-9]+: [0-9a-fA-F]* +1 +OBJECT +GLOBAL +DEFAULT +[0-9]+ +foo@FOO
 #...
 [      ]*[0-9]+: [0-9a-fA-F]* +1 +OBJECT +GLOBAL +DEFAULT +[0-9]+ +bar@@FOO
+#...
 [      ]*[0-9]+: [0-9a-fA-F]* +1 +OBJECT +GLOBAL +DEFAULT +[0-9]+ +foo
+#...
 [      ]*[0-9]+: [0-9a-fA-F]* +1 +OBJECT +GLOBAL +DEFAULT +[0-9]+ +bar
 #pass
index f168cff15848df93bec8100c16a62916196a6d5d..b0e49ea8216a0cb6c7c073c9f997537d905555dd 100644 (file)
@@ -166,17 +166,15 @@ if [catch { set ofd [open "tmpdir/$test2.d" w] } x] {
     return
 }
 
-# too big for d10v and msp
+# too big for avr, d10v and msp
 # lack of fancy orphan section handling causes overlap on fr30 and iq2000
 if { ![istarget "d10v-*-*"]
+     && ![istarget "avr-*-*"]
      && ![istarget "msp*-*-*"]
      && ![istarget "fr30-*-*"] 
      && ![istarget "iq2000-*-*"] } {
     foreach sfile $sfiles { puts $ofd "#source: $sfile" }
-    if { [istarget "avr-*-*"] } then {
-       puts $ofd "#as: -mmcu=avr6"
-       puts $ofd "#ld: -mavr6"
-    } elseif { [istarget spu*-*-*] } {
+    if { [istarget spu*-*-*] } {
        puts $ofd "#ld: --local-store 0:0"
     } else {
        puts $ofd "#ld:"
index f78297b12af943fc830b7494669ce0bcb4ab25ff..9e77ef3a25100e08ad7d7cfd7708a65c88104e7a 100644 (file)
@@ -5,7 +5,9 @@
 
 #...
 [0-9a-f]+[     ][dD][  ]_*foo1_start
+#...
 [0-9a-f]+[     ]D[     ]_*foo1
+#...
 [0-9a-f]+[     ][dD][  ]_*foo1_end
 [0-9a-f]+[     ][dD][  ]_*foo2_start
 [0-9a-f]+[     ]D[     ]_*foo2
index a8b0e4a245c4f26e31dd4f03cd010ac16e0ecac2..5be56618b0d4d696e3bc81486ff084721877bfdd 100644 (file)
@@ -183,11 +183,15 @@ set plugin_tests [list \
     $testobjfiles $libs" "" "" "" {{ld plugin-28.d}} "main.x" ] \
     [list "plugin warning" "-plugin $plugin2_path -plugin-opt warning \
     $testobjfiles $libs" "" "" "" {{ld plugin-29.d}} "main.x" ] \
-    [list "PR ld/17973" "-plugin $plugin2_path -shared $regassilent \
-                       -plugin-opt add:tmpdir/pr17973.o \
-    tmpdir/dummy.o" "" "" "" {{readelf -sW pr17973.d}} "main.x" ] \
 ]
 
+if [check_shared_lib_support] {
+    lappend plugin_tests [list "PR ld/17973" "-plugin $plugin2_path -shared $regassilent \
+                       -plugin-opt add:tmpdir/pr17973.o \
+    tmpdir/dummy.o" "" "" "" {{readelf -sW pr17973.d}} "main.x" ]
+}
+
+
 set plugin_lib_tests [list \
     [list "plugin ignore lib" "-plugin $plugin_path $regclm \
                        $regas $regcln -plugin-opt claim:tmpdir/func.o \
index 0531339e76ea3ebd359f24f6ad571e5ed63c6b65..367b878916b9fe7d37112d2a3d53580dd0671ded 100644 (file)
@@ -64,6 +64,12 @@ if [istarget sh64*-*-elf] {
     set ldflags "-e _start -mshelf32 $ldflags"
 }
 
+if [istarget avr-*-*] {
+    # Make .text start at a non-zero address, as some tests expect
+    # valid symbols to have non-zero values.
+    set ldflags "--section-start=.text=0x2 $ldflags"
+}
+
 # If we don't have g++ for the target, mark all tests as untested.
 if { ![is_remote host] && [which $CXX] == 0 } {
     foreach testitem $seltests {