[Bug gold/21503] New: Gold doesn't create linker stub symbols on ppc64

2017-05-18 Thread anton at samba dot org
https://sourceware.org/bugzilla/show_bug.cgi?id=21503

Bug ID: 21503
   Summary: Gold doesn't create linker stub symbols on ppc64
   Product: binutils
   Version: 2.29 (HEAD)
Status: UNCONFIRMED
  Severity: normal
  Priority: P2
 Component: gold
  Assignee: ccoutant at gmail dot com
  Reporter: anton at samba dot org
CC: ian at airs dot com
  Target Milestone: ---

The BFD linker creates symbols for each linker stub it creates. This is very
helpful for both debugging and performance analysis.

This was discovered when profiling an application with perf on Linux, and
finding a large chunk of samples that weren't getting resolved to a symbol.

-- 
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/21251] Support $SYSROOT in ld -L and INPUT command

2017-05-18 Thread nickc at redhat dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=21251

Nick Clifton  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

--- Comment #4 from Nick Clifton  ---
(In reply to Rainer Orth from comment #2)

Hi Rainer,

> Yes, works like a charm, thanks.

Great - I have checked the patch in.

> Btw., do you have any idea how widespread the use of '=' for the sysroot
> prefix is?

Hmm, I was going to say not a lot, but then I remembered that GCC uses it:

  https://gcc.gnu.org/onlinedocs/gcc/Directory-Options.html

So maybe it is more widespread than we realise.

> Besides, while were at --sysroot, did you have a chance to have a look
> at PR ld/21250.

I looked at it, shuddered, and looked away. :-}  I suspect that that PR
will turn out to be a can of worms, so I was going to treat it as low 
priority unless other people notice and complain too.

Cheers
  Nick

-- 
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/21251] Support $SYSROOT in ld -L and INPUT command

2017-05-18 Thread cvs-commit at gcc dot gnu.org
https://sourceware.org/bugzilla/show_bug.cgi?id=21251

--- Comment #3 from cvs-commit at gcc dot gnu.org  ---
The master branch has been updated by Nick Clifton :

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

commit 3aa2d05a728216bbb99dbb5718be9bb36429cf41
Author: Nick Clifton 
Date:   Thu May 18 15:07:59 2017 +0100

Treat a prefix of "$SYSROOT" in the same way as "=" when parsing linker
search paths.

PR ld/21251
* ldfile.c (ldfile_add_library_path): If the path starts with
$SYSROOT then use the sysroot as the real prefix.
* ldlang.c (lang_add_input_file): Treat $SYSROOT in the same
way as =.
* ldlex.l: Add $SYSROOT as allow prefix for a filename.
* ld.texinfo (-L): Document that $SYSROOT acts like = when
prefixing a library search path.
(INPUT): Likewise.
* testsuite/ld-scripts/sysroot-prefix.exp: Add $SYSROOT prefix
tests.

-- 
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/20882] GNU ld discards sections required by relocations in .debug_info with --gc-sections

2017-05-18 Thread hjl.tools at gmail dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=20882

--- Comment #13 from H.J. Lu  ---
We may also keep debug sections in otherwise unused files if there
is a .note.gnu.property section.

-- 
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/20882] GNU ld discards sections required by relocations in .debug_info with --gc-sections

2017-05-18 Thread amodra at gmail dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=20882

--- Comment #12 from Alan Modra  ---
The crx and xgate FAILs are due to broken relocation processing.  I don't think
they should be swept under the rug.  HJ has already pointed out why mips is
failing in https://sourceware.org/ml/binutils/2017-05/msg00195.html.  I've
looked into the arm failure, which is due to .ARM.attributes being marked KEEP
and then  _bfd_elf_gc_mark_extra_sections setting some_kept.  A possible fix is
to only set some_kept when an *alloc* section is kept.  Testing that now.

-- 
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/21251] Support $SYSROOT in ld -L and INPUT command

2017-05-18 Thread ro at CeBiTec dot Uni-Bielefeld.DE
https://sourceware.org/bugzilla/show_bug.cgi?id=21251

--- Comment #2 from Rainer Orth  ---
Hi Nick,

sorry for the very long delay in replying ;-(

Yes, works like a charm, thanks.

Btw., do you have any idea how widespread the use of '=' for the sysroot
prefix is?  It's kinda hard to search for -L = ...  I've only noticed it
very recently.  There's work going on to implement --sysroot in Solaris
ld, and the question poses itself if there's much point in supporting
the '=' form or just going for $SYSROOT here.

Besides, while were at --sysroot, did you have a chance to have a look
at PR ld/21250.

Thanks a lot.

Rainer

-- 
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/20882] GNU ld discards sections required by relocations in .debug_info with --gc-sections

2017-05-18 Thread nickc at redhat dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=20882

Nick Clifton  changed:

   What|Removed |Added

 CC||nickc at redhat dot com

--- Comment #11 from Nick Clifton  ---
Hi H.J.

  Unfortunately your new test fails for a number of targets:

FAIL: --gc-sections with relocations in debug section

  arm-none-eabi
  xgate-elf
  mipsisa32el-linux
  mips-sgi-irix6
  crx-elf
  mipsel-linux-gnu
  tic6x-elf
  tx39-elf
  mips64-linux
  xtensa-elf
  armeb-eabi
  msp430-elf
  v850-elf
  mips-elf
  arm-netbsdelf

Please could you have a look and see if you can fix/xfail them ?

Cheers
  Nick

-- 
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/20882] GNU ld discards sections required by relocations in .debug_info with --gc-sections

2017-05-18 Thread rguenther at suse dot de
https://sourceware.org/bugzilla/show_bug.cgi?id=20882

--- Comment #10 from rguenther at suse dot de ---
On Wed, 17 May 2017, hjl.tools at gmail dot com wrote:

> https://sourceware.org/bugzilla/show_bug.cgi?id=20882
> 
> H.J. Lu  changed:
> 
>What|Removed |Added
> 
>  Status|NEW |RESOLVED
>  Resolution|--- |FIXED
>Target Milestone|--- |2.29
> 
> --- Comment #8 from H.J. Lu  ---
> Fixed for 2.29.

Thanks HJ!

-- 
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/20882] GNU ld discards sections required by relocations in .debug_info with --gc-sections

2017-05-18 Thread cvs-commit at gcc dot gnu.org
https://sourceware.org/bugzilla/show_bug.cgi?id=20882

--- Comment #9 from cvs-commit at gcc dot gnu.org  ---
The master branch has been updated by Alan Modra :

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

commit 087ea5435ab5800e6c29671acab40dc6ca82
Author: Alan Modra 
Date:   Thu May 18 14:28:17 2017 +0930

pr20882 testcase

-gdwarf-sections doesn't work on targets that lack support to emit
address size relative relocs.  The testcase as it was avoided
-gdwarf-sections doing anything by providing a non-empty .debug_line.
It's better to not use -gdwarf-sections.  Also, the testcase failed
to match the output for 16-bit address targets like avr.

PR ld/20882
* testsuite/ld-gc/pr20882.d: Don't pass -gdwarf-sections to gas.
Allow for 16-bit address targets and match expected data fully.
* testsuite/ld-gc/pr20882a.s: Delete .debug_line section.
* testsuite/ld-gc/pr20882b.s: Likewise.
* testsuite/ld-gc/pr20882c.s: 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