espressif/binutils-gdb
1
0
Fork 0
mirror of https://github.com/espressif/binutils-gdb.git synced 2025-06-24 12:23:31 +08:00
Code Issues Packages Projects Releases Wiki Activity

PR ld/20828: Remove leading `_' from symbols used in tests

Browse Source 
Complement commit 81ff47b3a546 ("PR ld/20828: Fix linker script symbols
wrongly forced local with section GC") and remove the leading underscore
from `_fdata' and `_edata' symbols used in tests, fixing a:

FAIL: PR ld/20828 dynamic symbols with section GC (version script)

failure with targets such as: `bfin-elf', `bfin-uclinux', `metag-elf',
`metag-linux' `mn10300-elf', `sh-elf', `sh64-elf', and possibly other
ones, that have `_' set (with `elf_symbol_leading_char') as the leading
character for symbols.  As from commit 93252b1cf41a ("bfd/ld: handle ABI
prefixes in version scripts") these targets strip the leading underscore
before applying version script rules, because the (default) syntax for
symbol names is that of the C language rather than their low-level
symbol table encoding.

	ld/
	PR ld/20828
	* testsuite/ld-elf/pr20828.ld: Rename `_fdata' and `_edata' to
	`fdata' and `edata' respectively.
	* testsuite/ld-elf/pr20828.ver: Adjust accordingly.
	* testsuite/ld-elf/pr20828-a.sd: Likewise.
	* testsuite/ld-elf/pr20828-b.sd: Likewise.
	* testsuite/ld-elf/pr20828-c.sd: Likewise.
This commit is contained in:
Maciej W. Rozycki
2017-01-23 11:19:46 +00:00
parent adcbdc63e5
commit fda46c417f
6 changed files with 24 additions and 14 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
ld/ChangeLog
View File

@ -1,3 +1,13 @@
2017-01-23 Maciej W. Rozycki <macro@imgtec.com>
PR ld/20828
* testsuite/ld-elf/pr20828.ld: Rename `_fdata' and `_edata' to
`fdata' and `edata' respectively.
* testsuite/ld-elf/pr20828.ver: Adjust accordingly.
* testsuite/ld-elf/pr20828-a.sd: Likewise.
* testsuite/ld-elf/pr20828-b.sd: Likewise.
* testsuite/ld-elf/pr20828-c.sd: Likewise.
2017-01-23 Maciej W. Rozycki <macro@imgtec.com> 2017-01-23 Maciej W. Rozycki <macro@imgtec.com>
PR ld/20828 PR ld/20828

8
ld/testsuite/ld-elf/pr20828-a.sd
View File

@ -1,9 +1,9 @@
# Make sure `_fdata' is global rather than local in the dynamic symbol table, # Make sure `fdata' is global rather than local in the dynamic symbol table,
# e.g.: # e.g.:
# Num: Value Size Type Bind Vis Ndx Name # Num: Value Size Type Bind Vis Ndx Name
# 1: 00000000 0 NOTYPE GLOBAL DEFAULT 1 _fdata # 1: 00000000 0 NOTYPE GLOBAL DEFAULT 1 fdata
# vs: # vs:
# 1: 00000000 0 NOTYPE LOCAL DEFAULT 1 _fdata # 1: 00000000 0 NOTYPE LOCAL DEFAULT 1 fdata
#... #...
*[0-9]+: +[0-9a-f]+ +0 +NOTYPE +GLOBAL +DEFAULT +[0-9]+ +_fdata *[0-9]+: +[0-9a-f]+ +0 +NOTYPE +GLOBAL +DEFAULT +[0-9]+ +fdata
#pass #pass

8
ld/testsuite/ld-elf/pr20828-b.sd
View File

@ -1,9 +1,9 @@
# Make sure `_edata' is global rather than local in the dynamic symbol table, # Make sure `edata' is global rather than local in the dynamic symbol table,
# e.g.: # e.g.:
# Num: Value Size Type Bind Vis Ndx Name # Num: Value Size Type Bind Vis Ndx Name
# 1: 00000000 0 NOTYPE GLOBAL DEFAULT 1 _edata # 1: 00000000 0 NOTYPE GLOBAL DEFAULT 1 edata
# vs: # vs:
# 1: 00000000 0 NOTYPE LOCAL DEFAULT 1 _edata # 1: 00000000 0 NOTYPE LOCAL DEFAULT 1 edata
#... #...
*[0-9]+: +[0-9a-f]+ +0 +NOTYPE +GLOBAL +DEFAULT +[0-9]+ +_edata *[0-9]+: +[0-9a-f]+ +0 +NOTYPE +GLOBAL +DEFAULT +[0-9]+ +edata
#pass #pass

6
ld/testsuite/ld-elf/pr20828-c.sd
View File

@ -1,7 +1,7 @@
# Make sure no `_fdata' is present in the dynamic symbol table, e.g.: # Make sure no `fdata' is present in the dynamic symbol table, e.g.:
# Num: Value Size Type Bind Vis Ndx Name # Num: Value Size Type Bind Vis Ndx Name
# 1: 00000000 0 NOTYPE LOCAL DEFAULT 1 _fdata # 1: 00000000 0 NOTYPE LOCAL DEFAULT 1 fdata
#failif #failif
#... #...
.+ +_fdata .+ +fdata
#pass #pass

4
ld/testsuite/ld-elf/pr20828.ld
View File

@ -2,9 +2,9 @@ SECTIONS
{ {
.data : .data :
{ {
_fdata = .; fdata = .;
*(.data) *(.data)
_edata = .; edata = .;
} }
.dynamic : { *(.dynamic) } .dynamic : { *(.dynamic) }
.hash : { *(.hash) } .hash : { *(.hash) }

2
ld/testsuite/ld-elf/pr20828.ver
View File

@ -1 +1 @@
{ global: _edata; local: *; }; { global: edata; local: *; };