This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [gofrontend-dev] [PATCH 1/2, libgo] Add reflection support to gccgo for ppc64, ppc64le in gcc 4.9
- From: Ian Lance Taylor <iant at golang dot org>
- To: "Lynn A. Boger" <laboger at linux dot vnet dot ibm dot com>
- Cc: Michael Hudson-Doyle <michael dot hudson at canonical dot com>, gcc-patches <gcc-patches at gcc dot gnu dot org>, "gofrontend-dev at googlegroups dot com" <gofrontend-dev at googlegroups dot com>
- Date: Wed, 14 Jan 2015 07:24:56 -0800
- Subject: Re: [gofrontend-dev] [PATCH 1/2, libgo] Add reflection support to gccgo for ppc64, ppc64le in gcc 4.9
- Authentication-results: sourceware.org; auth=none
- References: <54AC0154 dot 7020700 at linux dot vnet dot ibm dot com> <CAOyqgcXhF0c4gnirAnj4b+0iY9=W_yDCUq4Rmff69Lx9ShbGgQ at mail dot gmail dot com> <54AD623E dot 9050109 at linux dot vnet dot ibm dot com> <CAOyqgcVh4S5ob1aQmvxkPBGKX9f8hX+yiVmJq4UkhKKTHgnWtQ at mail dot gmail dot com> <54AD6C37 dot 2060804 at linux dot vnet dot ibm dot com> <CAOyqgcXrwLD44o58hRCB16pdjd85DBETNhk_LSrs9kb8x-s=qg at mail dot gmail dot com> <87egr6p0le dot fsf at canonical dot com> <CAOyqgcU1aqbi3j+PqkOxLSubWObBm=peRPEWOQspFysikvoVaQ at mail dot gmail dot com> <54AE89A8 dot 2060702 at linux dot vnet dot ibm dot com> <CAOyqgcV6_u+Dt=-JZQhBKyHfZu8jkFQ+p_TQfwBrBd=wLEY-VA at mail dot gmail dot com> <54B673DF dot 2020002 at linux dot vnet dot ibm dot com>
On Wed, Jan 14, 2015 at 5:49 AM, Lynn A. Boger
<laboger@linux.vnet.ibm.com> wrote:
> I have an updated patch for the reflection functions in gcc 4.9 for ppc64 &
> ppc64le which allows the recover.go test to pass and the updated version of
> libgo/go/reflect/all_tests.go to pass once all testcases are enabled for
> ppc64 & ppc64le.
>
> Based on the discussion we've had on this patch, has the decision changed
> concerning whether or not this patch might be accepted into the gcc 4.9
> branch? If so, I will post the updated patch.
To me it is not appropriate to add brand new code, code that has never
been in mainline, to an existing release branch. That is not what
release branches are for. Release branches are for backports of bug
fixes, not for new code.
I assume that the people who want this code are not going to sit on
their hands waiting for the 4.9.3 release, which won't happen for
several months even in the best case. Why not provide this code as a
separate patch on top of the 4.9 releases? Or manage your own branch
if necessary?
Ian
- References:
- [PATCH 1/2, libgo] Add reflection support to gccgo for ppc64, ppc64le in gcc 4.9
- Re: [gofrontend-dev] [PATCH 1/2, libgo] Add reflection support to gccgo for ppc64, ppc64le in gcc 4.9
- Re: [gofrontend-dev] [PATCH 1/2, libgo] Add reflection support to gccgo for ppc64, ppc64le in gcc 4.9
- Re: [gofrontend-dev] [PATCH 1/2, libgo] Add reflection support to gccgo for ppc64, ppc64le in gcc 4.9
- Re: [gofrontend-dev] [PATCH 1/2, libgo] Add reflection support to gccgo for ppc64, ppc64le in gcc 4.9
- Re: [gofrontend-dev] [PATCH 1/2, libgo] Add reflection support to gccgo for ppc64, ppc64le in gcc 4.9
- Re: [gofrontend-dev] [PATCH 1/2, libgo] Add reflection support to gccgo for ppc64, ppc64le in gcc 4.9
- From: Michael Hudson-Doyle
- Re: [gofrontend-dev] [PATCH 1/2, libgo] Add reflection support to gccgo for ppc64, ppc64le in gcc 4.9
- Re: [gofrontend-dev] [PATCH 1/2, libgo] Add reflection support to gccgo for ppc64, ppc64le in gcc 4.9
- Re: [gofrontend-dev] [PATCH 1/2, libgo] Add reflection support to gccgo for ppc64, ppc64le in gcc 4.9
- Re: [gofrontend-dev] [PATCH 1/2, libgo] Add reflection support to gccgo for ppc64, ppc64le in gcc 4.9