ld: tests: add -msim when testing bfin targets
authorMike Frysinger <vapier@gentoo.org>
Thu, 9 Dec 2010 17:06:34 +0000 (12:06 -0500)
committerMike Frysinger <vapier@gentoo.org>
Thu, 14 Jan 2021 06:09:48 +0000 (01:09 -0500)
The Blackfin ELF compiler requires the user to explicitly select a CPU
target else it will fail:
bfin-elf-gcc: error: no processor type specified for linking

Select the sim target for these tests since we should (hopefully) have
access to the simulator.  At least, it's more likely than having access
to a real development board.

This makes the pass/fail numbers increase by a lot:
-# of expected passes        398
-# of unexpected failures    6
+# of expected passes        587
+# of unexpected failures    109

It looks like the vast majority of new failures are due to our omission
of COPY relocations:
  /* Bfin does not currently have a COPY reloc.  */
  if ((h->root.u.def.section->flags & SEC_ALLOC) != 0)
    {
      _bfd_error_handler (_("the bfin target does not currently support the generation of copy relocations"));
      return FALSE;
    }

There doesn't seem to be a way to easily disable tests that cause copy
relocations though, lets just take the hit for now.

* testsuite/config/default.exp [bfin*-elf*] (gcc_B_opt): Append -msim.

ld/ChangeLog
ld/testsuite/config/default.exp

index 60a31032b77b3e2724558cd3178983b861df1e1c..cbcda0a4112b44e04ab6178a36b66c1766a49fb2 100644 (file)
@@ -1,3 +1,7 @@
+2021-01-14  Mike Frysinger  <vapier@gentoo.org>
+
+       * testsuite/config/default.exp [bfin*-elf*] (gcc_B_opt): Add -msim.
+
 2021-01-14  Alan Modra  <amodra@gmail.com>
 
        PR 27160
index af313bb7dc690ce2151957331dbeff93f1937d12..4090f1287c468b2f105e5cd374c4cc5f8c5b0d3b 100644 (file)
@@ -157,6 +157,11 @@ if { [istarget rx-*-*] } {
     set ASFLAGS "-muse-conventional-section-names"
 }
 
+# Blackfin ELF targets require selection of an explicit CPU.  Use the sim.
+if {[istarget bfin*-elf*]} {
+    append gcc_B_opt " -msim"
+}
+
 # load the utility procedures
 load_lib ld-lib.exp