This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Four jit backports to gcc 5 branch
- From: David Malcolm <dmalcolm at redhat dot com>
- To: Basile Starynkevitch <basile at starynkevitch dot net>
- Cc: jit at gcc dot gnu dot org, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
- Date: Wed, 01 Jul 2015 10:49:23 -0400
- Subject: Re: Four jit backports to gcc 5 branch
- Authentication-results: sourceware.org; auth=none
- References: <1435592671 dot 13727 dot 136 dot camel at surprise> <20150630064302 dot GA3804 at ovh dot starynkevitch dot net> <1435669264 dot 13727 dot 213 dot camel at surprise>
On Tue, 2015-06-30 at 09:01 -0400, David Malcolm wrote:
> On Tue, 2015-06-30 at 08:43 +0200, Basile Starynkevitch wrote:
> > On Mon, Jun 29, 2015 at 11:44:31AM -0400, David Malcolm wrote:
> > > I've gone over the changes to the gcc/jit and gcc/testsuite/jit.dg
> > > directories in trunk since gcc 5 and backported the following 4 changes
> > > from trunk to the gcc-5-branch:
> > >
> >
> >
> > I'll be delighted if switch statements ability would be backported to
> > GCC 5. (Its absence is IMHO a severe bug in GCCJIT, but perhaps GCC rules
> > forbid backporting new features, even when they correct a huge
> > deficiency and when the patch adding them is probably quite small).
>
> I don't agree with your characterization of the severity of this, but I
> do think we ought to fix it if possible.
>
> I don't know of any rules about what is and what isn't suitable for
> backporting. The jit work as a whole is relatively new and has been
> messaged as "experimental".
>
> I'm the jit maintainer, and I believe the patch is reasonable to
> backport: it doesn't break API or ABI (although it extends it), it has
> docs and test coverage, and doesn't touch anything outside of the jit
> (beyond adding the new header gcc/typed-splay-tree.h, which is only used
> by the jit).
>
> Hence I intend to backport it to gcc-5-branch, but I've got to get it
> into trunk first :) (hopefully later today)
Switch statements for the jit are now in gcc-5-branch, as of r225255.