Fix unresolved test in binutils for pdp11.

* binutils/testsuite/binutils-all/pr25662-pdp11.s: Alternate source file
for test using section pseudo-ops compatible with pdp11.
* binutils/testsuite/binutils-all/objcopy.exp: Select alternate source.

diff --git a/binutils/ChangeLog b/binutils/ChangeLog
index 10806a9353009d40bac7f4ac8dda0dab849d94fe..9bbebf561949c4afd9de7c2b2554a862477336d8 100644 (file)
--- a/binutils/ChangeLog
+++ b/binutils/ChangeLog
@@ -1,3 +1,9 @@
+2020-06-04  Stephen Casner  <casner@acm.org>
+
+       * testsuite/binutils-all/pr25662-pdp11.s: Alternate source file
+       for test using section pseudo-ops compatible with pdp11.
+       * testsuite/binutils-all/objcopy.exp: Select alternate source.
+
 2020-06-04  Jose E. Marchesi  <jose.marchesi@oracle.com>
 
        * objdump.c (disassemble_data): Set disasm_info.endian_code to

diff --git a/binutils/testsuite/binutils-all/objcopy.exp b/binutils/testsuite/binutils-all/objcopy.exp
index 56a7db8199b7a375b9ca5a44657e893e9488d952..dd74860f9e9823bdeffeeeab57868fb0508c52ec 100644 (file)
--- a/binutils/testsuite/binutils-all/objcopy.exp
+++ b/binutils/testsuite/binutils-all/objcopy.exp
@@ -1348,4 +1348,10 @@ setup_xfail "sh-*-coff*"
 setup_xfail "spu-*-*"
 clear_xfail "hppa*64*-*-hpux*" "hppa*-*-linux*" "hppa*-*-lites*"
 clear_xfail "hppa*-*-*n*bsd*" "hppa*-*-rtems*" "hppa*-*-*elf*"
-objcopy_test "pr25662" pr25662.s executable "" "-T$srcdir/$subdir/pr25662.ld"
+if { [istarget pdp11-*-*] } {
+    set src "pr25662-pdp11.s"
+} else {
+    set src "pr25662.s"
+}
+
+objcopy_test "pr25662" $src executable "" "-T$srcdir/$subdir/pr25662.ld"

diff --git a/binutils/testsuite/binutils-all/pr25662-pdp11.s b/binutils/testsuite/binutils-all/pr25662-pdp11.s
new file mode 100644 (file)
index 0000000..0b2481e
--- /dev/null
+++ b/binutils/testsuite/binutils-all/pr25662-pdp11.s
@@ -0,0 +1,36 @@
+/* PR 25662: objcopy sets invalid sh_offset for the first section in a
+   no_contents segment containing program headers.
+
+   Several conditions are required for the bug to manifest:
+   - The first loadable segment (which contains the program headers) must only
+     contain SHT_NOBITS sections. .bss is the SHT_NOBITS section in this test.
+   - The next loadable segment must have a !SHT_NOBITS loadable section. .data
+     is the !SHT_NOBITS section in this test.
+   - .bss must be positioned after .data in the executable file itself.
+   - The size of .data must be such that the calculated VMA of the .bss
+     section that follows it is not congruent with the file offset of .bss,
+     modulo the p_align of its segment, i.e.:
+       (VMA(.data) + sizeof(.data)) % (.bss_segment.p_align) != 0
+     This will force the sh_offset of .bss to be aligned so it appears within
+     .data.
+   - The size of .data must be larger than the program headers in the first
+     loadable segment, so that the file offset of .bss is immediately
+     after .data, and not padded to a valid alignment by the program headers.
+
+   The bug originally only manifested for ELF targets, but there's no reason not
+   to run this testcase for other file formats.  This variant source for pdp11
+   uses .text rather than .section text, etc., because the latter are not
+   supported,  */
+
+       .bss
+a:
+       .zero   0x2
+
+       .data
+c:
+       .zero   0x201
+
+       .text
+       .global _start
+_start:
+       .long 0