This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] [PING] [PR libitm/61164] Remove redefinition of glibc internal macro __always_inline
- From: Torvald Riegel <triegel at redhat dot com>
- To: Gleb Fotengauer-Malinovskiy <glebfm at altlinux dot org>
- Cc: Jakub Jelinek <jakub at redhat dot com>, gcc-patches at gcc dot gnu dot org
- Date: Fri, 21 Aug 2015 11:35:12 +0200
- Subject: Re: [PATCH] [PING] [PR libitm/61164] Remove redefinition of glibc internal macro __always_inline
- Authentication-results: sourceware.org; auth=none
- References: <20150506145455 dot GC3470 at glebfm dot cloud dot tilaa dot com> <1431695067 dot 25070 dot 3 dot camel at triegel dot csb> <20150611113609 dot GD10323 at glebfm dot cloud dot tilaa dot com> <1439746517 dot 6240 dot 4 dot camel at localhost dot localdomain> <20150817111645 dot GB18970 at glebfm dot cloud dot tilaa dot com>
On Mon, 2015-08-17 at 13:16 +0200, Gleb Fotengauer-Malinovskiy wrote:
> On Sun, Aug 16, 2015 at 07:35:17PM +0200, Torvald Riegel wrote:
> > On Thu, 2015-06-11 at 14:36 +0300, Gleb Fotengauer-Malinovskiy wrote:
> > > On Fri, May 15, 2015 at 03:04:27PM +0200, Torvald Riegel wrote:
> > > > On Wed, 2015-05-06 at 17:54 +0300, Gleb Fotengauer-Malinovskiy wrote:
> > > > > 2015-05-06 Gleb Fotengauer-Malinovskiy <glebfm@altlinux.org>
> > > > >
> > > > > PR libitm/61164
> > > > > * local_atomic (__always_inline): Rename to...
> > > > > (__libitm_always_inline): ... this.
> > > >
> > > > OK. Thanks.
> > >
> > > You are welcome.
> > >
> > > It seems still not applied, AFAICS.
> >
> > I forgot to ask you at Cauldron whether you have completed a copyright
> > assignment agreement?
>
> No, I didn't. I'm in progress.
>
> > I don't think this is small enough to be a
> > trivial patch.
>
> "A regular series of repeated changes, such as renaming a symbol, is not
> legally significant even if the symbol has to be renamed in many places."
> https://www.gnu.org/prep/maintain/html_node/Legally-Significant.html#Legally-Significant
OK. I committed this to trunk for you, with the "(tiny change)" marker
in the ChangeLog as suggested on this GNU web page.