Re: [PATCH] docs: update symver attribute description

2021-04-12 Thread Martin Liška
On 4/12/21 3:50 PM, H.J. Lu wrote: > GCC failed to bootstrap: > > /export/gnu/import/git/sources/gcc/gcc/doc/extend.texi:3870: misplaced { > /export/gnu/import/git/sources/gcc/gcc/doc/extend.texi:3872: misplaced } > /export/gnu/import/git/sources/gcc/gcc/doc/extend.texi:3874: unknown > command `VE

Re: [PATCH] docs: update symver attribute description

2021-04-12 Thread H.J. Lu via Gcc-patches
On Mon, Apr 12, 2021 at 5:54 AM Martin Liška wrote: > > On 4/12/21 2:18 PM, Jakub Jelinek wrote: > > On Mon, Apr 12, 2021 at 02:15:16PM +0200, Martin Liška wrote: > >> The old syntax with the alias is quite ugly.. > > > > Not that much. And users have no other option (besides inline asm > > but t

Re: [PATCH] docs: update symver attribute description

2021-04-12 Thread Jakub Jelinek via Gcc-patches
point! Is it fine now? Ok, thanks. > >From 6dda0ec10a1b0c60e6e9afe7fc45370d0132b5e3 Mon Sep 17 00:00:00 2001 > From: Martin Liska > Date: Mon, 12 Apr 2021 13:42:33 +0200 > Subject: [PATCH] docs: update symver attribute description > > gcc/ChangeLog: > > * doc/extend.t

Re: [PATCH] docs: update symver attribute description

2021-04-12 Thread Martin Liška
fuse users. For symbol versions which just a single symbol they > don't need any aliases... Very good point! Is it fine now? Thanks, Martin > > Jakub > >From 6dda0ec10a1b0c60e6e9afe7fc45370d0132b5e3 Mon Sep 17 00:00:00 2001 From: Martin Liska Date: Mon, 12 Apr 2021 13:4

Re: [PATCH] docs: update symver attribute description

2021-04-12 Thread Jakub Jelinek via Gcc-patches
On Mon, Apr 12, 2021 at 02:32:35PM +0200, Martin Liška wrote: > +If you have an older release of binutils release, then symbol alias needs to s/binutils release/binutils/ > +be used: > + > +@smallexample > +__attribute__ ((__symver__ ("foo@@VERS_2"))) > +__attribute__ ((alias ("foo_v1"))) > +int

Re: [PATCH] docs: update symver attribute description

2021-04-12 Thread Martin Liška
ight, so something like this? Thanks, Martin >From 750b715225d480fcb74e765623d54acc42ac25e3 Mon Sep 17 00:00:00 2001 From: Martin Liska Date: Mon, 12 Apr 2021 13:42:33 +0200 Subject: [PATCH] docs: update symver attribute description gcc/ChangeLog: * doc/extend.texi: Be more precise in docu

Re: [PATCH] docs: update symver attribute description

2021-04-12 Thread Jakub Jelinek via Gcc-patches
On Mon, Apr 12, 2021 at 02:15:16PM +0200, Martin Liška wrote: > The old syntax with the alias is quite ugly.. Not that much. And users have no other option (besides inline asm but that doesn't work with LTO well). > > so that people who don't have gcc configured against > > binutils 2.35 or newe

Re: [PATCH] docs: update symver attribute description

2021-04-12 Thread Martin Liška
On 4/12/21 1:50 PM, Jakub Jelinek wrote: > On Mon, Apr 12, 2021 at 01:44:54PM +0200, Martin Liška wrote: >> This improves documentation as noticed by Jakub. >> >> Ready for master? >> Thanks, >> Martin >> >> gcc/ChangeLog: >> >> * doc/extend.texi: Be more precise in documentation >> of sy

Re: [PATCH] docs: update symver attribute description

2021-04-12 Thread Jakub Jelinek via Gcc-patches
On Mon, Apr 12, 2021 at 01:44:54PM +0200, Martin Liška wrote: > This improves documentation as noticed by Jakub. > > Ready for master? > Thanks, > Martin > > gcc/ChangeLog: > > * doc/extend.texi: Be more precise in documentation > of symver attribute. Ok, but I'd prefer to see the o

[PATCH] docs: update symver attribute description

2021-04-12 Thread Martin Liška
This improves documentation as noticed by Jakub. Ready for master? Thanks, Martin gcc/ChangeLog: * doc/extend.texi: Be more precise in documentation of symver attribute. --- gcc/doc/extend.texi | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/gcc/doc/ex