https://sourceware.org/bugzilla/show_bug.cgi?id=19803
Alan Modra changed:
What|Removed |Added
Resolution|--- |FIXED
Status|ASSIGNED
https://sourceware.org/bugzilla/show_bug.cgi?id=19803
--- Comment #22 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=a061de07e441718d3658be332fd3172d87c7440b
commit a061de07e441718d3658be332f
https://sourceware.org/bugzilla/show_bug.cgi?id=19803
--- Comment #21 from martin.koegler at chello dot at ---
(In reply to Nick Clifton from comment #19)
> OK - this revised patch contains an attempt to fix this function as well.
> Please could you give it a try and let me know the results.
Ple
https://sourceware.org/bugzilla/show_bug.cgi?id=19803
--- Comment #20 from martin.koegler at chello dot at ---
(In reply to Nick Clifton from comment #19)
> > In my option, this change is not necessary, as it just would hides broken
> > symbols in the export library.
>
> Actually I think that it
https://sourceware.org/bugzilla/show_bug.cgi?id=19803
Nick Clifton changed:
What|Removed |Added
Attachment #9104|0 |1
is obsolete|
https://sourceware.org/bugzilla/show_bug.cgi?id=19803
--- Comment #18 from martin.koegler at chello dot at ---
First, 41f46ed9fea1a066de95b6a85c56393beef0b8b8 (Fix possible failure in the
AVR linker tests) introduced (propably accidentically) did a change to
pe_dll_generate_implib not documented i
https://sourceware.org/bugzilla/show_bug.cgi?id=19803
--- Comment #17 from Nick Clifton ---
Created attachment 9115
--> https://sourceware.org/bugzilla/attachment.cgi?id=9115&action=edit
patch to add _ prefixed exported names
Hi Martin,
OK - I have checked the current patch in.
Does this
https://sourceware.org/bugzilla/show_bug.cgi?id=19803
--- Comment #16 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=4153b6dbb0f38a16fd5b583761aa811212fbb9a5
commit 4153b6dbb0f38a16fd5b583761
https://sourceware.org/bugzilla/show_bug.cgi?id=19803
--- Comment #15 from martin.koegler at chello dot at ---
The remaining i686-w64-mingw32 problem:
It uses a different symbol name different to the exported symbol name:
Eg. testval is exported, but the symbol in the .o file is called _testval.
https://sourceware.org/bugzilla/show_bug.cgi?id=19803
--- Comment #14 from martin.koegler at chello dot at ---
You can consider my xstrdup use as a bug [safest choice].
Binutils 2.26 + patch from Comment #13 works for a larger project for
x86_64-w32-mingw.
For i686-w64-mingw32, I have an issue r
https://sourceware.org/bugzilla/show_bug.cgi?id=19803
--- Comment #9 from martin.koegler at chello dot at ---
I have reworked the first version a little bit.
The DEF-handling looks OK. SEH prevents the removal of a function,
Get dx.s from Comment #2.
t1.s:
=
.file "t1.c"
https://sourceware.org/bugzilla/show_bug.cgi?id=19803
--- Comment #6 from martin.koegler at chello dot at ---
The last patch is incorrect. In my testcase, it still exports the removed
variable in the DLL export table:
[Ordinal/Name Pointer] Table
[ 0] testval
So directly linking with th
https://sourceware.org/bugzilla/show_bug.cgi?id=19803
Nick Clifton changed:
What|Removed |Added
Status|NEW |ASSIGNED
--- Comment #7 from Nick Clif
https://sourceware.org/bugzilla/show_bug.cgi?id=19803
--- Comment #13 from martin.koegler at chello dot at ---
Created attachment 9108
--> https://sourceware.org/bugzilla/attachment.cgi?id=9108&action=edit
Cleaned patch
I'm currently testings this on a tree without
41f46ed9fea1a066de95b6a85c563
https://sourceware.org/bugzilla/show_bug.cgi?id=19803
Nick Clifton changed:
What|Removed |Added
Attachment #9096|0 |1
is obsolete|
https://sourceware.org/bugzilla/show_bug.cgi?id=19803
--- Comment #8 from martin.koegler at chello dot at ---
Created attachment 9106
--> https://sourceware.org/bugzilla/attachment.cgi?id=9106&action=edit
Patch V1
--
You are receiving this mail because:
You are on the CC list for the bug.
https://sourceware.org/bugzilla/show_bug.cgi?id=19803
--- Comment #11 from Nick Clifton ---
Created attachment 9107
--> https://sourceware.org/bugzilla/attachment.cgi?id=9107&action=edit
Patch VII
Hi Martin,
I like the patch, but I have one question: why did you need to use xstrdup to
make
https://sourceware.org/bugzilla/show_bug.cgi?id=19803
--- Comment #10 from martin.koegler at chello dot at ---
(In reply to Nick Clifton from comment #7)
> True - but it looks to me like the data structures for generating the export
> table are set up far too early - ie before garbage collection -
https://sourceware.org/bugzilla/show_bug.cgi?id=19803
--- Comment #12 from martin.koegler at chello dot at ---
Looks OK, but I have troubles applying it.
Additionally, there is 41f46ed9fea1a066de95b6a85c56393beef0b8b8 on master,
which looks like an earlier version.
I'll try to test it.
--
You
https://sourceware.org/bugzilla/show_bug.cgi?id=19803
--- Comment #4 from martin.koegler at chello dot at ---
Fixes all my testcases.
--
You are receiving this mail because:
You are on the CC list for the bug.
___
bug-binutils mailing list
bug-binutils
https://sourceware.org/bugzilla/show_bug.cgi?id=19803
--- Comment #3 from Nick Clifton ---
Created attachment 9096
--> https://sourceware.org/bugzilla/attachment.cgi?id=9096&action=edit
Proposed patch
Hi Martin,
Please could you try out this patch. It is a work in progress, but it might
do
https://sourceware.org/bugzilla/show_bug.cgi?id=19803
--- Comment #2 from martin.koegler at chello dot at ---
The bug is caused by deleting sections and still exporting their symbols with
bogus values:
dx.s:
=
.text
.globl DllMainCRTStartup
DllMainCRTStartup:
https://sourceware.org/bugzilla/show_bug.cgi?id=19803
Nick Clifton changed:
What|Removed |Added
CC||nickc at redhat dot com
--- Comment #1
23 matches
Mail list logo
