This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Fix PR66848 by enforcing 16-bit alignment on darwin
- From: Mike Stump <mikestump at comcast dot net>
- To: Jack Howarth <howarth dot at dot gcc at gmail dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>, Iain Sandoe <iain at codesourcery dot com>, hboehm at gcc dot gnu dot org
- Date: Mon, 28 Dec 2015 13:44:18 -0800
- Subject: Re: [PATCH] Fix PR66848 by enforcing 16-bit alignment on darwin
- Authentication-results: sourceware.org; auth=none
- References: <CAJMcOU9Pwr0xJYQvd1XMvHUrbGBYwiTAp_eNSKBKknGxMciF3w at mail dot gmail dot com>
On Dec 22, 2015, at 9:08 AM, Jack Howarth <howarth.at.gcc@gmail.com> wrote:
> This bug doesn't exist in the more recent boehm-gc 7.2 or
> later releases. Until the exact change from 6.6 to 7.2 that suppresses
> this bug is identified or FSF gcc's boehm-gc is rebased on the 7.2
> version or later, the simple fix to suppress this issue on darwin is
> to enforce 16-bit alignment in boehm-gc/include/private/gcconfig.h for
> that target.
It would be nice to just pull in 7.2 (or just the latest release)… seems like that should be easier to maintain to me.
> Okay for gcc trunk and back ports to gcc-5-branch and gcc-4_9-branch?
I’d like to punt the approval to a libboehm person, though, not sure we have any. Just checked we don’t have an official maintainer listed. :-( Do we have anybody that wants to review the patch or should I? Though I would prefer a new import of a newer library to fix the issue, I’m tempted to just approve the patch. Kinda don’t want to as any deviation makes importing a new library annoying. What do others think?