[Bug gas/13709] New: symbol definition loop not detected - stackdump

2012-02-19 Thread sourceware.org at neonics dot com
http://sourceware.org/bugzilla/show_bug.cgi?id=13709

 Bug #: 13709
   Summary: symbol definition loop not detected - stackdump
   Product: binutils
   Version: 2.22
Status: NEW
  Severity: normal
  Priority: P2
 Component: gas
AssignedTo: unassig...@sourceware.org
ReportedBy: sourceware@neonics.com
Classification: Unclassified


GNU assembler version 2.22.51 (i686-cygwin) using BFD version (GNU Binutils)
2.2
2.51.20111013


The message 'symbol definition loop encountered at SYMBOL' is not
shown, and a stackdump occurs, under the following conditions:

1) .intel_syntax noprefix
2) the symbol is used in an instruction

Example source:

.equ BUG, BUG + 1
.intel_syntax noprefix
mov ax, BUG


Expected output:
---
bug.s: error: symbol definition loop encountered at `Bug' (though a line number
would be useful)

What happens instead:
--
-rw--- 1 Kenney None 0 Feb 19 16:31 a.out
-rwx-- 1 Kenney None 0 Feb 19 16:31 as.exe.stackdump*
--

(stackdump file is empty)


When the symbol is not referenced, the warning occurs. 
Also, not reproducible when using at&t syntax.

-- 
Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email
--- 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/13710] New: [Regression] .def files containing LIBRARY statement can't be parsed any more

2012-02-19 Thread erik-sources-redhat-bugzilla at vanpienbroek dot nl
http://sourceware.org/bugzilla/show_bug.cgi?id=13710

 Bug #: 13710
   Summary: [Regression] .def files containing LIBRARY statement
can't be parsed any more
   Product: binutils
   Version: 2.23 (HEAD)
Status: NEW
  Severity: normal
  Priority: P2
 Component: ld
AssignedTo: unassig...@sourceware.org
ReportedBy: erik-sources-redhat-bugzi...@vanpienbroek.nl
CC: ktiet...@googlemail.com
Classification: Unclassified
Target: i686-w64-mingw32, x86_64-w64-mingw32


In the last couple of days a change was applied to binutils which makes the
behaviour of the processing of .def files more strict. Due to this change some
libraries fail to compile, for example cairo.

The .def file which is used by cairo contains something along the lines of
this:

EXPORTS
my_func
LIBRARY mylib.dll

This used to work fine with binutils 2.22.51.20120113, but when using binutils
2.22.52.20120216 this results in the following error:

/usr/lib64/gcc/i686-w64-mingw32/4.7.0/../../../../i686-w64-mingw32/bin/ld:
a.def:3: syntax error
/usr/lib64/gcc/i686-w64-mingw32/4.7.0/../../../../i686-w64-mingw32/bin/ld:a.def:
file format not recognized; treating as linker script
/usr/lib64/gcc/i686-w64-mingw32/4.7.0/../../../../i686-w64-mingw32/bin/ld:a.def:2:
syntax error
collect2: error: ld returned 1 exit status

After some discussion with Kai Tietz we came to the conclusion that these type
of .def files are invalid according to MSDN:
http://msdn.microsoft.com/en-us/library/d91k01sh%28v=vs.80%29.aspx

This documentation indicates that .def files must start with the 'LIBRARY
xxx.dll' statement followed by the 'EXPORTS' statement, so in this regard the
more strict behaviour of binutils is correct.

So I tried fixing cairo so that it uses a .def file which looks like this:

LIBRARY mylib.dll
EXPORTS
my_func

While doing so I noticed that libtool also tries to do some funny things with
the .def file:

libtool: link: if test "x`/bin/sed 1q .libs/libcairo.def`" = xEXPORTS; then cp
.libs/libcairo.def .libs/libcairo-2.dll.def; else echo EXPORTS >
.libs/libcairo-2.dll.def; cat .libs/libcairo.def >> .libs/libcairo-2.dll.def;
fi

It looks like libtool always checks the first line of the given .def file. If
this first line isn't 'EXPORTS' then it will be automatically added to the
resulting .def file. Given the .def file above, the resulting .def file will
become:

EXPORTS
LIBRARY mylib.dll
EXPORTS
my_func

This .def file isn't accepted by binutils as well.

So to fix all this, changes have to be applied to both cairo and libtool.
However, the annoying thing with libtool is that every libtool-using piece of
software is bundling their own copy of libtool so it will take quite some time
before a patched version of libtool is really in use. While I agree that
binutils should be as compliant and strict as possible I think that the damage
here is worse than the benefits. So I would like to suggest that this change in
behaviour in binutils is reverted to the old behaviour (for now) and that we
also try to get this fixed properly in libtool, cairo and perhaps several other
pieces of software.

-- 
Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email
--- 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/13710] [Regression] .def files containing LIBRARY statement can't be parsed any more

2012-02-19 Thread erik-sources-redhat-bugzilla at vanpienbroek dot nl
http://sourceware.org/bugzilla/show_bug.cgi?id=13710

--- Comment #1 from Erik van Pienbroek  2012-02-19 22:46:44 UTC ---
Filed bugs for libtool and cairo:
libtool: https://savannah.gnu.org/support/index.php?107959
cairo: https://bugs.freedesktop.org/show_bug.cgi?id=46299

-- 
Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email
--- 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