[PATCH] ld/ELF: correct tidying of sec64k object files
Jan Beulich
jbeulich@suse.com
Wed Dec 3 09:29:39 GMT 2025
These consume quite a bit of space (especially noticeable during batch
testing), but are rather mechanical to (re)create. In 2.28, when the
object file names used were changed (from dump<N>.o to names derived from
source file names), the tidying code wasn't updated (and hence lost its
effect).
---
Additionally it looks wasteful to build half of (all the same) object
files twice: The first (ld -r) test's outputs could be re-used rather
than re-generating them; only the latter half would need assembling in
addition. Sadly there doesn't look to be a way to specify object files
to (re)use, as we have only #source to specify inputs. Which would
apparently leave playing with ldflags.
--- a/ld/testsuite/ld-elf/sec64k.exp
+++ b/ld/testsuite/ld-elf/sec64k.exp
@@ -243,6 +243,6 @@ if { ![istarget "d10v-*-*"]
}
# Tidy up.
-for { set i 1 } { $i < $max_sec / $secs_per_file } { incr i } {
- catch "exec rm -f tmpdir/dump$i.o" status
+for { set i 0 } { $i < $max_sec / $secs_per_file } { incr i } {
+ catch "exec rm -f tmpdir/sec64-$i.o" status
}
More information about the Binutils
mailing list