Launchpad has imported 14 comments from the remote bug at
http://sourceware.org/bugzilla/show_bug.cgi?id=10856.

If you reply to an imported comment from within Launchpad, your comment
will be sent to the remote bug automatically. Read more about
Launchpad's inter-bugtracker facilities at
https://help.launchpad.net/InterBugTracking.

------------------------------------------------------------------------
On 2009-10-27T23:54:55+00:00 Matthias Klose wrote:

The patch to fix PR gas/10704 causes sha.cpp assembled in a way that causes the
sha-256 test not to terminate. Undoing the patch runs the test fine. Replacing
the sha.o with one assembled with gas having this patch reverted lets the test 
pass.

Reply at: https://bugs.launchpad.net/binutils/+bug/461303/comments/17

------------------------------------------------------------------------
On 2009-10-27T23:55:37+00:00 Matthias Klose wrote:

Created attachment 4329
diff of disassembled files

Reply at: https://bugs.launchpad.net/binutils/+bug/461303/comments/18

------------------------------------------------------------------------
On 2009-10-27T23:56:18+00:00 Matthias Klose wrote:

Created attachment 4330
sha .s and .o files

Reply at: https://bugs.launchpad.net/binutils/+bug/461303/comments/19

------------------------------------------------------------------------
On 2009-10-28T02:53:32+00:00 Alan Modra wrote:

Simpler testcase:

 .text
 .intel_syntax noprefix
 .space 5480
 .byte 0,0,0
0:
 .space 1620
 .byte 0
1:
 .space 2468
 nop
 jne 1b
 .space 52
 nop
 jl 0b

Without the .intel_syntax this assembles OK.


Reply at: https://bugs.launchpad.net/binutils/+bug/461303/comments/20

------------------------------------------------------------------------
On 2009-10-28T08:21:59+00:00 Cvs-commit wrote:

Subject: Bug 10856

CVSROOT:        /cvs/src
Module name:    src
Changes by:     [email protected]   2009-10-28 08:21:45

Modified files:
        gas            : ChangeLog expr.c 
        gas/testsuite  : ChangeLog 
        gas/testsuite/gas/i386: intelpic.d 

Log message:
        PR gas/10856
        * expr.c (resolve_expression): Only add "left" value to O_symbol
        expression when the symbol is undefined and different from the
        original symbol.  Simplify negative logic.
        
        * gas/i386/intelpic.d: Correct.

Patches:
http://sources.redhat.com/cgi-bin/cvsweb.cgi/src/gas/ChangeLog.diff?cvsroot=src&r1=1.3999&r2=1.4000
http://sources.redhat.com/cgi-bin/cvsweb.cgi/src/gas/expr.c.diff?cvsroot=src&r1=1.78&r2=1.79
http://sources.redhat.com/cgi-bin/cvsweb.cgi/src/gas/testsuite/ChangeLog.diff?cvsroot=src&r1=1.1573&r2=1.1574
http://sources.redhat.com/cgi-bin/cvsweb.cgi/src/gas/testsuite/gas/i386/intelpic.d.diff?cvsroot=src&r1=1.3&r2=1.4


Reply at: https://bugs.launchpad.net/binutils/+bug/461303/comments/21

------------------------------------------------------------------------
On 2009-10-28T08:24:03+00:00 Cvs-commit wrote:

Subject: Bug 10856

CVSROOT:        /cvs/src
Module name:    src
Branch:         binutils-2_20-branch
Changes by:     [email protected]   2009-10-28 08:23:49

Modified files:
        gas            : ChangeLog expr.c 

Log message:
        PR gas/10856
        * expr.c (resolve_expression): Only add "left" value to O_symbol
        expression when the symbol is undefined and different from the
        original symbol.  Simplify negative logic.

Patches:
http://sources.redhat.com/cgi-bin/cvsweb.cgi/src/gas/ChangeLog.diff?cvsroot=src&only_with_tag=binutils-2_20-branch&r1=1.3938.2.27&r2=1.3938.2.28
http://sources.redhat.com/cgi-bin/cvsweb.cgi/src/gas/expr.c.diff?cvsroot=src&only_with_tag=binutils-2_20-branch&r1=1.77.2.1&r2=1.77.2.2


Reply at: https://bugs.launchpad.net/binutils/+bug/461303/comments/22

------------------------------------------------------------------------
On 2009-10-28T10:03:48+00:00 Matthias Klose wrote:

checked that the sha test passes, when sha.o is built with this patch.


Reply at: https://bugs.launchpad.net/binutils/+bug/461303/comments/23

------------------------------------------------------------------------
On 2009-10-28T10:08:48+00:00 Alan Modra wrote:

.

Reply at: https://bugs.launchpad.net/binutils/+bug/461303/comments/24

------------------------------------------------------------------------
On 2009-11-05T12:14:29+00:00 Alan Modra wrote:

*** Bug 10906 has been marked as a duplicate of this bug. ***

Reply at: https://bugs.launchpad.net/binutils/+bug/461303/comments/31

------------------------------------------------------------------------
On 2009-12-13T17:55:22+00:00 David-Sarah Hopwood wrote:

Any indication of when this fix is likely to get into a release? The more OS 
distributions as 2.20 goes into, the more problems this bug will cause.

Reply at: https://bugs.launchpad.net/binutils/+bug/461303/comments/32

------------------------------------------------------------------------
On 2009-12-13T21:00:41+00:00 Zooko O'Whielacronx wrote:

This launchpad page tracks the increasing number of OS distributions and open
source projects that are being negatively affected by the bug in 2.20:

https://bugs.launchpad.net/fedora/+source/binutils/+bug/461303

Reply at: https://bugs.launchpad.net/binutils/+bug/461303/comments/33

------------------------------------------------------------------------
On 2009-12-14T09:04:48+00:00 Gingold wrote:

Subject: Re:  [2.20 regression] gas creates wrong code which results in
a test failure in libcrypto++'s sha2 test


On Dec 13, 2009, at 6:55 PM, david-sarah at jacaranda dot org wrote:

> 
> ------- Additional Comments From david-sarah at jacaranda dot org  2009-12-13 
> 17:55 -------
> Any indication of when this fix is likely to get into a release? The more OS 
> distributions as 2.20 goes into, the more problems this bug will cause.

I agree that this is annoying.  What about a 2.20.1 release in January ?

Tristan.


Reply at: https://bugs.launchpad.net/binutils/+bug/461303/comments/34

------------------------------------------------------------------------
On 2010-07-06T04:44:56+00:00 Zooko O'Whielacronx wrote:

Hey folks, some distributions such as MinGW have still not patched their 
version of binutils or reverted 
to binutils 2.19:

https://sourceforge.net/tracker/index.php?func=detail&aid=2913876&group_id=2435&atid=102435

This is still causing problems for my users:

http://tahoe-lafs.org/trac/tahoe-lafs/ticket/853

If you folks could go ahead and release a binutil 2.20.1 then that might 
stimulate distributors like 
MinGW to upgrade to it and that would fix the problem for my users. Thanks!

Reply at: https://bugs.launchpad.net/binutils/+bug/461303/comments/35

------------------------------------------------------------------------
On 2010-12-19T18:18:07+00:00 Zooko O'Whielacronx wrote:

binutils 2.20.1, released 2010-03-03, does *not* have the ChangeLog
entry from http://sourceware.org/bugzilla/show_bug.cgi?id=10856#c5 but
*does* have the patch to expr.c. Weird. But I guess it is fixed in
binutils 2.20.1.

Reply at: https://bugs.launchpad.net/binutils/+bug/461303/comments/37


** Changed in: binutils
   Importance: Unknown => Medium

** Bug watch added: Tahoe-LAFS Trac #853
   http://tahoe-lafs.org/trac/tahoe-lafs/ticket/853

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/461303

Title:
  generates-bad-code regression

-- 
ubuntu-bugs mailing list
[email protected]
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to