[Bug binutils/23595] "simple objcopy of executable" failure for msp430-elf with -mlarge

2018-09-03 Thread cvs-commit at gcc dot gnu.org
https://sourceware.org/bugzilla/show_bug.cgi?id=23595

--- Comment #3 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=2542e49e21e6b7270f72c7be0fc4ff1a986371da

commit 2542e49e21e6b7270f72c7be0fc4ff1a986371da
Author: Jozef Lawrynowicz 
Date:   Mon Sep 3 11:04:05 2018 +0930

PR23595, simple objcopy of executable failure for msp430-elf

VMA of the first section in the segment containing the ELF file header
(and possibly section headers too) can't be used to reliably find the
size of the headers plus padding.  What's really needed is sh_offset
of the first section assuming it has contents (vma does have a
relationship to sh_offset, but is only guaranteed in demand paged
executables).

If the first section is SHT_NOBITS and it hasn't been converted to
have file contents by the existence of a following SHT_PROGBITS
section in the same segment, the sh_offset value also isn't reliable.

PR 23595
elf.c (copy_elf_program_header): When first segment contains
only the headers and SHT_NOBITS sections, use segment p_filesz
to calculate header and padding size.  Use filepos of the first
section otherwise.

-- 
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/23591] undefined reference to `__start___sancov_cntrs'

2018-09-03 Thread cvs-commit at gcc dot gnu.org
https://sourceware.org/bugzilla/show_bug.cgi?id=23591

--- Comment #11 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=0d9a433be406220413671495a9b0fab577a48b5e

commit 0d9a433be406220413671495a9b0fab577a48b5e
Author: Alan Modra 
Date:   Mon Sep 3 10:49:57 2018 +0930

Re: ld: Lookup section in output with the same name

Fixes pr23591 test failures on hppa64-hpux and score-elf, and xfails
frv-linux and lm32-linux.

PR ld/23591
* testsuite/ld-elf/pr23591a.s,
* testsuite/ld-elf/pr23591b.s,
* testsuite/ld-elf/pr23591c.s: Don't start directives in first column.
* testsuite/ld-elf/pr23591.d: xfail frv-linux and lm32-linux.
Allow __start___sancov_cntrs as a local 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 binutils/23595] "simple objcopy of executable" failure for msp430-elf with -mlarge

2018-09-03 Thread amodra at gmail dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=23595

Alan Modra  changed:

   What|Removed |Added

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

--- Comment #4 from Alan Modra  ---
Patch applied

-- 
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 gas/23570] AVR .noinit section defaults to PROGBITS used to be NOBITS

2018-09-03 Thread nickc at redhat dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=23570

Nick Clifton  changed:

   What|Removed |Added

 CC||nickc at redhat dot com

--- Comment #1 from Nick Clifton  ---
Created attachment 11226
  --> https://sourceware.org/bugzilla/attachment.cgi?id=11226&action=edit
Proposed patch

Hi Gabriel,

  Please could you try out this patch and let me know if it does
  what you want.

  Are there any other special sections for the AVR that will need
  similar handling ?

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/23425] unresolved symbol diagnostic ends up calling find_abstract_instance with relocations applied causing spurious 'DWARF error: invalid abstract instance DIE ref'

2018-09-03 Thread hjl.tools at gmail dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=23425

--- Comment #9 from H.J. Lu  ---
(In reply to Jon Grant from comment #5)
> There's a testcase for the first attached. Can that be worked on first?

No, I need your testcase as well unless it is the same as the one here.

-- 
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/23425] unresolved symbol diagnostic ends up calling find_abstract_instance with relocations applied causing spurious 'DWARF error: invalid abstract instance DIE ref'

2018-09-03 Thread hjl.tools at gmail dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=23425

--- Comment #10 from H.J. Lu  ---
Created attachment 11227
  --> https://sourceware.org/bugzilla/attachment.cgi?id=11227&action=edit
A patch

Please try this.

-- 
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 binutils/23589] A hang problem for c++filt

2018-09-03 Thread nickc at redhat dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=23589

Nick Clifton  changed:

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
 CC||nickc at redhat dot com
 Resolution|--- |MOVED

--- Comment #1 from Nick Clifton  ---
Hi Peng,

  You are correct, this is a problem with the C++ name demangling code.
  Unfortunately you have reported it to the wrong place.  Although c++filt
  is part of the binutils project, it gets its C++ demangling facilities
  from the libiberty library, which actually part of the gcc project.

  So please could you refile this bug report here:

https://gcc.gnu.org/bugzilla/enter_bug.cgi?product=gcc

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 binutils/23601] Memory leak dwarf.c

2018-09-03 Thread nickc at redhat dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=23601

Nick Clifton  changed:

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
 CC||nickc at redhat dot com
 Resolution|--- |WONTFIX

--- Comment #2 from Nick Clifton  ---
Hi Tianxiao,

  Thanks very much for reporting this problem.

  Unfortunately memory leaks like this are not something that we are
  worried about with this project.  They will all be resolved when
  the objdump program exists, and it is not a program that stays in
  memory, so the leaks will never become significant.

  This whilst we acknowledge that there are leaks, we are not going
  to fix 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 gas/23570] AVR .noinit section defaults to PROGBITS used to be NOBITS

2018-09-03 Thread rauter.gabriel at gmail dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=23570

--- Comment #2 from Gabriel Rauter  ---
Hi Nick,

yes the patch fixes the issue and the .noinit section gets correctly set to
NOBITS. Sadly i can not comment on other special sections since i only
investigated this issue since it broke a popular custom keyboard firmware when
using avr-gcc 8.2. https://github.com/qmk/qmk_firmware/issues/3657

But as far as i can tell the .noinit section is the only one who needs to be
set to NOBITS since the section is part of the .bss section.
https://www.nongnu.org/avr-libc/user-manual/mem_sections.html

I am quiet unsure why gcc stopped setting NOBITS for the .noinit section. They
still warn you when you try to initialize a variable set to the .noinit section
tough. And it looks like they set the bss flags even commenting it should be
nobits
https://gcc.gnu.org/viewcvs/gcc/trunk/gcc/config/avr/avr.c?revision=257301&view=markup#l10322.

Thanks for your help
Gabriel

-- 
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 gas/23570] AVR .noinit section defaults to PROGBITS used to be NOBITS

2018-09-03 Thread cvs-commit at gcc dot gnu.org
https://sourceware.org/bugzilla/show_bug.cgi?id=23570

--- 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=8744470deabb6a2fae35be0f66c60ee4ba094c1d

commit 8744470deabb6a2fae35be0f66c60ee4ba094c1d
Author: Nick Clifton 
Date:   Mon Sep 3 17:06:05 2018 +0100

Change the .section directive for the AVR assembler so that the .noinit
section is always given the ELF NOBITS section type.

PR gas/23570
* config/tc-avr.c (md_pseudo_table): Add entry for "secction".
(avr_set_section): New function.  Ensures that the .noinit section
gets the NOBITS ELF section type.

-- 
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 gas/23570] AVR .noinit section defaults to PROGBITS used to be NOBITS

2018-09-03 Thread nickc at redhat dot com
https://sourceware.org/bugzilla/show_bug.cgi?id=23570

Nick Clifton  changed:

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
 Resolution|--- |FIXED

--- Comment #4 from Nick Clifton  ---
Hi Gabriel,

  OK - I have applied the patch.  If you come across any more sections
  that do need special handling, please feel free to reopen this PR.

  I do not know why the change was made in gcc-land, but I suspect that
  the reasoning was that "the assembler is the right place for special
  sections to be handled in special ways".  Or something like that.

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/23425] unresolved symbol diagnostic ends up calling find_abstract_instance with relocations applied causing spurious 'DWARF error: invalid abstract instance DIE ref'

2018-09-03 Thread jg at jguk dot org
https://sourceware.org/bugzilla/show_bug.cgi?id=23425

--- Comment #11 from Jon Grant  ---
(In reply to H.J. Lu from comment #10)
> Created attachment 11227 [details]
> A patch
> 
> Please try this.

Hi H.J.
Could you attach you Ld binary please? I'm on x64 Ubuntu

-- 
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