This is the mail archive of the
libstdc++@gcc.gnu.org
mailing list for the libstdc++ project.
Re: [Patch ARM v2] Fix PR target/56846
- From: Jonathan Wakely <jwakely at redhat dot com>
- To: Tony Wang <tony dot wang at arm dot com>
- Cc: Ramana Radhakrishnan <Ramana dot Radhakrishnan at arm dot com>, gcc-patches <gcc-patches at gcc dot gnu dot org>, libstdc++ at gcc dot gnu dot org
- Date: Wed, 10 Sep 2014 15:38:25 +0100
- Subject: Re: [Patch ARM v2] Fix PR target/56846
- Authentication-results: sourceware.org; auth=none
- References: <001a01cfcc13$115e2060$341a6120$ at arm dot com> <20140909103947 dot GD22778 at redhat dot com> <001c01cfccca$97039c10$c50ad430$ at arm dot com>
On 10/09/14 15:41 +0800, Tony Wang wrote:
-----Original Message-----
From: Jonathan Wakely [mailto:jwakely@redhat.com]
Sent: Tuesday, September 09, 2014 6:40 PM
To: Tony Wang
Cc: Ramana Radhakrishnan; gcc-patches; libstdc++@gcc.gnu.org
Subject: Re: [Patch ARM v2] Fix PR target/56846
On 09/09/14 17:47 +0800, Tony Wang wrote:
>Hi there,
>
>This is a updated patch to fix pr56846. Bootstrapped on x86_64-unknown-linux-gnu and no regression for
>regression test on Linux and a cross regression test on arm-none-eabi.
>
>OK for the trunk?
>
>gcc/libstdc++-v3/ChangeLog:
>2014-09-09 Tony Wang <tony.wang@arm.com>
>
> PR target/56846
> * libsupc++/eh_personality.cc(PERSONALITY_FUNCTION):
^
There should be a space after the file name (look at existing
changelog entries)
OK with that change.
Committed with your comment on trunk, and also tested on 4.9 branch with no regression. So is it ok to back
port to 4.9 branch?
I'd at least like to see it working OK on trunk for a while without
regressions, and I don't know the EABI code well enough to be
confident it's safe to backport.
So in this case I'd request approval from one of the Release Managers
and/or ARM maintainers.