[Bug debug/68229] .debug_pubnames length field is too large

2017-08-03 Thread egallager at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68229

Eric Gallager  changed:

   What|Removed |Added

URL||https://bugzilla.redhat.com
   ||/show_bug.cgi?id=1278872
 Resolution|FIXED   |MOVED

[Bug debug/68229] .debug_pubnames length field is too large

2017-08-03 Thread todd.allen at ccur dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68229

--- Comment #6 from Todd Allen  ---
They released a patch last year:

https://bugzilla.redhat.com/show_bug.cgi?id=1278872

[Bug debug/68229] .debug_pubnames length field is too large

2017-08-03 Thread egallager at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68229

Eric Gallager  changed:

   What|Removed |Added

 Status|WAITING |RESOLVED
 CC||egallager at gcc dot gnu.org
 Resolution|--- |FIXED

--- Comment #5 from Eric Gallager  ---
(In reply to Todd Allen from comment #4)
> It appears to have been fixed in gcc-4.9.0 by Sterling Augustine,
> 2013-07-25, with the new include_pubname_in_output function.  I don't have a
> 4.9.0 compiler, but I did test it with gcc-4.9.2 on Fedora 21, and that
> worked.  I'll take up the issue with RedHat, since it's RHEL 7.0 that
> provides gcc-4.8.x still.

OK, closing as FIXED, but if you have a link to the issue you opened with
RedHat, it could be closed as MOVED instead.

[Bug debug/68229] .debug_pubnames length field is too large

2015-11-06 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68229

Richard Biener  changed:

   What|Removed |Added

   Keywords||wrong-debug
 Status|UNCONFIRMED |WAITING
   Last reconfirmed||2015-11-06
 Ever confirmed|0   |1

--- Comment #3 from Richard Biener  ---
Works for me with GCC 5 (?), so can you please check a newer GCC as GCC 4.8 is
no longer supported.

readelf -S progtest.o libtest.o progtest.x | grep -A1 .debug_pubnames
  [ 9] .debug_pubnames   PROGBITS   01da
   0028     0 0 1
--
  [ 9] .debug_pubnames   PROGBITS   01ee
   0025     0 0 1
--
  [29] .debug_pubnames   PROGBITS   1230
   004d     0 0 1
readelf -wp progtest.o libtest.o

File: progtest.o
Contents of the .debug_pubnames section:

  Length:  36
  Version: 2
  Offset into .debug_info section: 0x0
  Size of area in .debug_info section: 218

Offset  Name
80  main
c4  variable


File: libtest.o
Contents of the .debug_pubnames section:

  Length:  33
  Version: 2
  Offset into .debug_info section: 0x0
  Size of area in .debug_info section: 241

Offset  Name
a2  f
b9  i
dc  pt

readelf -wp progtest.x
Contents of the .debug_pubnames section:

  Length:  36
  Version: 2
  Offset into .debug_info section: 0x15b
  Size of area in .debug_info section: 218

Offset  Name
80  main
c4  variable
  Length:  33
  Version: 2
  Offset into .debug_info section: 0x235
  Size of area in .debug_info section: 241

Offset  Name
a2  f
b9  i
dc  pt

[Bug debug/68229] .debug_pubnames length field is too large

2015-11-06 Thread todd.allen at ccur dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68229

--- Comment #4 from Todd Allen  ---
It appears to have been fixed in gcc-4.9.0 by Sterling Augustine, 2013-07-25,
with the new include_pubname_in_output function.  I don't have a 4.9.0
compiler, but I did test it with gcc-4.9.2 on Fedora 21, and that worked.  I'll
take up the issue with RedHat, since it's RHEL 7.0 that provides gcc-4.8.x
still.

[Bug debug/68229] .debug_pubnames length field is too large

2015-11-05 Thread todd.allen at ccur dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68229

--- Comment #1 from Todd Allen  ---
Created attachment 36660
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=36660=edit
Patch to correct gcc 4.8.2

This patch was against gcc-4.8.2-16.2.el7_0 from RHEL 7.0.  But it should be
pretty clear how it applies to vanilla gcc.  In fact, it might just apply with
slight line number differences.

[Bug debug/68229] .debug_pubnames length field is too large

2015-11-05 Thread todd.allen at ccur dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68229

--- Comment #2 from Todd Allen  ---
Yeah, the patch code is clumsy.  I wrote it that way to mirror the use of a
continue in output_pubnames().