public inbox for gdb-testers@sourceware.org
help / color / mirror / Atom feed
* [binutils-gdb] PR24511, nm should not mark symbols in .init_array as "t"
@ 2019-05-04  8:16 sergiodj+buildbot
  2019-05-04  8:23 ` Failures on RHEL-s390x-m64, branch master sergiodj+buildbot
                   ` (6 more replies)
  0 siblings, 7 replies; 8+ messages in thread
From: sergiodj+buildbot @ 2019-05-04  8:16 UTC (permalink / raw)
  To: gdb-testers

*** TEST RESULTS FOR COMMIT a288c270991de1578ad28ac312120f4167347234 ***

Author: Alan Modra <amodra@gmail.com>
Branch: master
Commit: a288c270991de1578ad28ac312120f4167347234

PR24511, nm should not mark symbols in .init_array as "t"

This patch restricts the section names matched in coff_section_type,
a function that translates section names to symbol type, and arranges
to translate section flags to symbol type before looking at names.
The latter change resulted in various test failures due to improper
section flags being used in tests, and by the plugin support, so fix
that too.

The new test fails on many ELF targets that lack .init/fini_array
in their scripts.  I've just xfailed those.  pru-elf oddly defines
__init_array_begin rather than __init_array_start.  I've left that
target as a FAIL, and pj-elf too which reports an error for undefined
weak symbols.

bfd/
	PR 24511
	* syms.c (coff_section_type): Only allow '.', '$' and numeric
	following the standard section names.
	(bfd_decode_symclass): Prioritize section flag tests in
	decode_section_type before name tests in coff_section_type.
	* plugin.c (bfd_plugin_canonicalize_symtab): Init fake_section
	and fake_common_section using BFD_FAKE_SECTION.  Use "fake" as
	their names and choose standard .text section flags for
	fake_section.
ld/
	PR 24511
	* testsuite/ld-elf/pr14156a.d: Allow for .init/.fini being a
	data section on hppa64.
	* testsuite/ld-elf/pr14156b.d: Likewise.
	* testsuite/ld-scripts/pr18963.t: Map standard sections to set
	output section flags.
	* testsuite/ld-scripts/sane1.t: Likewise.
	* testsuite/ld-elf/init-fini-arrays.s: Reference __init_array_start
	and __fini_array_start.  Define __start et al.
	* testsuite/ld-elf/pr24511.d: New test.


^ permalink raw reply	[flat|nested] 8+ messages in thread
end of thread, other threads:[~2019-05-06 15:23 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-05-04  8:16 [binutils-gdb] PR24511, nm should not mark symbols in .init_array as "t" sergiodj+buildbot
2019-05-04  8:23 ` Failures on RHEL-s390x-m64, branch master sergiodj+buildbot
2019-05-04  9:13 ` Failures on Debian-s390x-m64, " sergiodj+buildbot
2019-05-04  9:54 ` Failures on Debian-s390x-native-extended-gdbserver-m64, " sergiodj+buildbot
2019-05-06 14:55 ` Failures on Fedora-x86_64-native-extended-gdbserver-m64, " sergiodj+buildbot
2019-05-06 15:20 ` Failures on Fedora-x86_64-native-gdbserver-m64, " sergiodj+buildbot
2019-05-06 15:23 ` Failures on Fedora-x86_64-m64, " sergiodj+buildbot
2019-05-06 15:33 ` Failures on Fedora-i686, " sergiodj+buildbot

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).