Launchpad has imported 8 comments from the remote bug at
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=45112.

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://documentation.ubuntu.com/launchpad/user/reference/bugs/multi-project-bugs/about-multi-project-bugs/#bugs-in-external-trackers.

------------------------------------------------------------------------
On 2010-07-28T18:24:18+00:00 Uweigand-gcc wrote:

Building the following testcase fails with G++ 4.5 and later.
G++ 4.4 works fine.


struct JSString
{
  unsigned char mLength;
  static JSString unitStringTable[];
};

JSString JSString::unitStringTable[] __attribute__ ((aligned (8))) = { 1
};

int bug [__alignof__ (JSString::unitStringTable) >= 8 ? 1 : -1];


The test case is reduced from Mozilla, where the bug sometimes causes the 
JavaScript interpreter to crash.  See also:
https://bugzilla.mozilla.org/show_bug.cgi?id=582593

The problem appears to be that cp-decl.c:duplicate_decls fails to merge
the DECL_USER_ALIGN flag from the definition into the declaration.

This bug was introduced by the following patch:
http://gcc.gnu.org/ml/gcc-patches/2009-06/msg00763.html

Before that patch, the DECL_USER_ALIGN flag was part of a block copied
in whole via memcpy by duplicate_decls.  The patch moved that flag to
another location outside that block, so it is no longer copied ...

Reply at: https://bugs.launchpad.net/gcc-linaro/+bug/604874/comments/26

------------------------------------------------------------------------
On 2010-07-28T21:47:11+00:00 Uweigand-gcc wrote:

Proposed fix posted here:
http://gcc.gnu.org/ml/gcc-patches/2010-07/msg02223.html

Reply at: https://bugs.launchpad.net/gcc-linaro/+bug/604874/comments/30

------------------------------------------------------------------------
On 2010-07-30T15:50:00+00:00 Uweigand-gcc wrote:

Subject: Bug 45112

Author: uweigand
Date: Fri Jul 30 15:49:34 2010
New Revision: 162716

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=162716
Log:
gcc/
        PR c++/45112
        * cp/decl.c (duplicate_decls): Merge DECL_USER_ALIGN and DECL_PACKED.

gcc/testsuite/
        PR c++/45112
        * testsuite/g++.dg/pr45112.C: New test.

Added:
    trunk/gcc/testsuite/g++.dg/pr45112.C
Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/cp/decl.c
    trunk/gcc/testsuite/ChangeLog


Reply at: https://bugs.launchpad.net/gcc-linaro/+bug/604874/comments/31

------------------------------------------------------------------------
On 2010-07-30T16:19:38+00:00 Uweigand-gcc wrote:

Fixed in mainline.  Will check in to 4.5 after 4.5.1 release.

Reply at: https://bugs.launchpad.net/gcc-linaro/+bug/604874/comments/32

------------------------------------------------------------------------
On 2010-07-31T15:46:31+00:00 Uweigand-gcc wrote:

Subject: Bug 45112

Author: uweigand
Date: Sat Jul 31 15:46:15 2010
New Revision: 162783

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=162783
Log:
gcc/
        PR c++/45112
        * cp/decl.c (duplicate_decls): Merge DECL_USER_ALIGN and DECL_PACKED.

gcc/testsuite/
        PR c++/45112
        * testsuite/g++.dg/pr45112.C: New test.

Added:
    branches/gcc-4_5-branch/gcc/testsuite/g++.dg/pr45112.C
Modified:
    branches/gcc-4_5-branch/gcc/ChangeLog
    branches/gcc-4_5-branch/gcc/cp/decl.c
    branches/gcc-4_5-branch/gcc/testsuite/ChangeLog


Reply at: https://bugs.launchpad.net/gcc-linaro/+bug/604874/comments/34

------------------------------------------------------------------------
On 2010-07-31T15:48:00+00:00 Uweigand-gcc wrote:

Fixed in 4.5 branch (for 4.5.2) as well.

Reply at: https://bugs.launchpad.net/gcc-linaro/+bug/604874/comments/35

------------------------------------------------------------------------
On 2010-07-31T17:43:01+00:00 Uweigand-gcc wrote:

Subject: Bug 45112

Author: uweigand
Date: Sat Jul 31 17:42:48 2010
New Revision: 162785

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=162785
Log:
Move PR c++/45112 ChangeLog entry to correct location.

Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/cp/ChangeLog


Reply at: https://bugs.launchpad.net/gcc-linaro/+bug/604874/comments/37

------------------------------------------------------------------------
On 2010-07-31T17:44:09+00:00 Uweigand-gcc wrote:

Subject: Bug 45112

Author: uweigand
Date: Sat Jul 31 17:43:59 2010
New Revision: 162786

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=162786
Log:
Move PR c++/45112 ChangeLog entry to correct location.

Modified:
    branches/gcc-4_5-branch/gcc/ChangeLog
    branches/gcc-4_5-branch/gcc/cp/ChangeLog


Reply at: https://bugs.launchpad.net/gcc-linaro/+bug/604874/comments/38

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

Title:
  firefox fails to build from source with Linaro toolchain

To manage notifications about this bug go to:
https://bugs.launchpad.net/firefox/+bug/604874/+subscriptions


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

Reply via email to