[Bug ld/23818] Symbols from discarded section in IR object leaked into final executable

2018-11-06 Thread cvs-commit at gcc dot gnu.org
https://sourceware.org/bugzilla/show_bug.cgi?id=23818

--- Comment #7 from cvs-commit at gcc dot gnu.org  ---
The binutils-2_31-branch branch has been updated by Nick Clifton
:

https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=84fdbf1a2c901965414eafc96e3bd15fc72ac123

commit 84fdbf1a2c901965414eafc96e3bd15fc72ac123
Author: Nick Clifton 
Date:   Tue Nov 6 16:06:52 2018 +

When assigning symbol version, we should hide debug symbols defined in
discarded sections from IR objects so that they can be removed later.

bfd PR ld/23818
* elflink.c (_bfd_elf_link_assign_sym_version): Hide symbols
defined in discarded input sections.

ld  * testsuite/ld-plugin/lto.exp: Run PR ld/23818 test.
* testsuite/ld-plugin/pr23818.d: New file.
* testsuite/ld-plugin/pr23818.t: Likewise.
* testsuite/ld-plugin/pr23818a.c: Likewise.
* testsuite/ld-plugin/pr23818b.c: Likewise.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug ld/23818] Symbols from discarded section in IR object leaked into final executable

2018-10-26 Thread hjl.tools at gmail dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=23818

--- Comment #6 from H.J. Lu  ---
(In reply to Romain Geissler from comment #5)
> Is it ok to backport in release branch 2.31 ?

Please request backport on the binutils mailing list.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug ld/23818] Symbols from discarded section in IR object leaked into final executable

2018-10-26 Thread romain.geissler at amadeus dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=23818

--- Comment #5 from Romain Geissler  ---
After rebuilding libssh2 and a few other open source libraries, now indeed I
don't see any more these odd absolute relocations, and now ld.lld happily
accepts them as valid input .so files.

So it looks like you fixed it ;)

Is it ok to backport in release branch 2.31 ?

Cheers,
Romain

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug ld/23818] Symbols from discarded section in IR object leaked into final executable

2018-10-24 Thread hjl.tools at gmail dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=23818

--- Comment #4 from H.J. Lu  ---
(In reply to Romain Geissler from comment #3)
> Thanks, I will check that this indeed fixes my case with libssh2.
> 
> I understand this problem is not really wide and might actually affect just
> me since I don't expect that using -ffat-lto-objects is very common, but do
> you think this commit deserves to be backported on branch 2.31, or do you
> prefer I patch it manually on my side until 2.32 is out ?

Please verify that it does fix your problem first.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug ld/23818] Symbols from discarded section in IR object leaked into final executable

2018-10-24 Thread romain.geissler at amadeus dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=23818

--- Comment #3 from Romain Geissler  ---
Thanks, I will check that this indeed fixes my case with libssh2.

I understand this problem is not really wide and might actually affect just me
since I don't expect that using -ffat-lto-objects is very common, but do you
think this commit deserves to be backported on branch 2.31, or do you prefer I
patch it manually on my side until 2.32 is out ?

Cheers,
Romain

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug ld/23818] Symbols from discarded section in IR object leaked into final executable

2018-10-24 Thread hjl.tools at gmail dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=23818

H.J. Lu  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED
   Target Milestone|--- |2.32

--- Comment #2 from H.J. Lu  ---
Fixed for 2.32.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils


[Bug ld/23818] Symbols from discarded section in IR object leaked into final executable

2018-10-24 Thread cvs-commit at gcc dot gnu.org
https://sourceware.org/bugzilla/show_bug.cgi?id=23818

--- Comment #1 from cvs-commit at gcc dot gnu.org  ---
The master branch has been updated by H.J. Lu :

https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=0a640d719684f25bdb88ae60148c6978e6131701

commit 0a640d719684f25bdb88ae60148c6978e6131701
Author: H.J. Lu 
Date:   Wed Oct 24 18:08:13 2018 -0700

ELF: Hide symbols defined in discarded input sections

When assigning symbol version, we should hide debug symbols defined in
discarded sections from IR objects so that they can be removed later.

bfd/

PR ld/23818
* elflink.c (_bfd_elf_link_assign_sym_version): Hide symbols
defined in discarded input sections.

ld/

PR ld/23818
* testsuite/ld-plugin/lto.exp: Run PR ld/23818 test.
* testsuite/ld-plugin/pr23818.d: New file.
* testsuite/ld-plugin/pr23818.t: Likewise.
* testsuite/ld-plugin/pr23818a.c: Likewise.
* testsuite/ld-plugin/pr23818b.c: Likewise.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils