Re: [PATCH 4/4] define ASM_OUTPUT_LABEL to the name of a function

2015-08-07 Thread Richard Sandiford
Richard Biener writes: >>> E.g. for the attached microbenchmark I get: >>> >>> Time taken, normalised to VERSION==1 >>> >>> VERSION==1: 1.000 >>> VERSION==2: 1.377 >>> VERSION==3: 3.202 (1.638 with -minline-all-stringops) >>> VERSION==4: 4.242 (2.921 with -minline-all-stringops) >>

Re: [PATCH 4/4] define ASM_OUTPUT_LABEL to the name of a function

2015-08-07 Thread Richard Biener
On August 7, 2015 3:50:33 PM GMT+02:00, Trevor Saunders wrote: >On Fri, Aug 07, 2015 at 10:45:57AM +0100, Richard Sandiford wrote: >> Trevor Saunders writes: >> > On Thu, Aug 06, 2015 at 08:36:36PM +0100, Richard Sandiford wrote: >> >> An integrated assembler or tighter asm output would be nice,

Re: [PATCH 4/4] define ASM_OUTPUT_LABEL to the name of a function

2015-08-07 Thread Trevor Saunders
On Fri, Aug 07, 2015 at 10:45:57AM +0100, Richard Sandiford wrote: > Trevor Saunders writes: > > On Thu, Aug 06, 2015 at 08:36:36PM +0100, Richard Sandiford wrote: > >> An integrated assembler or tighter asm output would be nice, but when > >> I last checked LLVM was usually faster than GCC even w

Re: [PATCH 4/4] define ASM_OUTPUT_LABEL to the name of a function

2015-08-07 Thread Richard Sandiford
Trevor Saunders writes: > On Thu, Aug 06, 2015 at 08:36:36PM +0100, Richard Sandiford wrote: >> An integrated assembler or tighter asm output would be nice, but when >> I last checked LLVM was usually faster than GCC even when compiling to asm, >> even though LLVM does use indirection (in the form

Re: [PATCH 4/4] define ASM_OUTPUT_LABEL to the name of a function

2015-08-06 Thread Trevor Saunders
On Thu, Aug 06, 2015 at 08:36:36PM +0100, Richard Sandiford wrote: > David Malcolm writes: > > On Wed, 2015-08-05 at 16:22 -0400, Trevor Saunders wrote: > >> On Wed, Aug 05, 2015 at 11:34:28AM -0400, David Malcolm wrote: > >> > On Wed, 2015-08-05 at 11:28 -0400, David Malcolm wrote: > >> > > On We

Re: [PATCH 4/4] define ASM_OUTPUT_LABEL to the name of a function

2015-08-06 Thread Richard Sandiford
David Malcolm writes: > On Wed, 2015-08-05 at 16:22 -0400, Trevor Saunders wrote: >> On Wed, Aug 05, 2015 at 11:34:28AM -0400, David Malcolm wrote: >> > On Wed, 2015-08-05 at 11:28 -0400, David Malcolm wrote: >> > > On Wed, 2015-08-05 at 13:47 +0200, Richard Biener wrote: >> > > > On Wed, Aug 5, 2

Re: [PATCH 4/4] define ASM_OUTPUT_LABEL to the name of a function

2015-08-06 Thread David Malcolm
On Wed, 2015-08-05 at 16:22 -0400, Trevor Saunders wrote: > On Wed, Aug 05, 2015 at 11:34:28AM -0400, David Malcolm wrote: > > On Wed, 2015-08-05 at 11:28 -0400, David Malcolm wrote: > > > On Wed, 2015-08-05 at 13:47 +0200, Richard Biener wrote: > > > > On Wed, Aug 5, 2015 at 12:57 PM, Trevor Saund

Re: [PATCH 4/4] define ASM_OUTPUT_LABEL to the name of a function

2015-08-05 Thread Segher Boessenkool
On Wed, Aug 05, 2015 at 03:59:18PM +0200, Richard Biener wrote: > >> Makes it quite unparsable to editors of course ... > > > > The idea of having .S files that aren't reasonably editable seems kind > > of silly, but I guess its up to the gas people. > > Heh, indeed. Maybe instead do > > .insert

Re: [PATCH 4/4] define ASM_OUTPUT_LABEL to the name of a function

2015-08-05 Thread Trevor Saunders
On Wed, Aug 05, 2015 at 11:34:28AM -0400, David Malcolm wrote: > On Wed, 2015-08-05 at 11:28 -0400, David Malcolm wrote: > > On Wed, 2015-08-05 at 13:47 +0200, Richard Biener wrote: > > > On Wed, Aug 5, 2015 at 12:57 PM, Trevor Saunders > > > wrote: > > > > On Mon, Jul 27, 2015 at 11:06:58AM +020

Re: [PATCH 4/4] define ASM_OUTPUT_LABEL to the name of a function

2015-08-05 Thread David Malcolm
On Wed, 2015-08-05 at 11:28 -0400, David Malcolm wrote: > On Wed, 2015-08-05 at 13:47 +0200, Richard Biener wrote: > > On Wed, Aug 5, 2015 at 12:57 PM, Trevor Saunders > > wrote: > > > On Mon, Jul 27, 2015 at 11:06:58AM +0200, Richard Biener wrote: > > >> On Sat, Jul 25, 2015 at 4:37 AM, wrote:

Re: [PATCH 4/4] define ASM_OUTPUT_LABEL to the name of a function

2015-08-05 Thread David Malcolm
On Wed, 2015-08-05 at 13:47 +0200, Richard Biener wrote: > On Wed, Aug 5, 2015 at 12:57 PM, Trevor Saunders > wrote: > > On Mon, Jul 27, 2015 at 11:06:58AM +0200, Richard Biener wrote: > >> On Sat, Jul 25, 2015 at 4:37 AM, wrote: > >> > From: Trevor Saunders > >> > > >> > * config/arc/

Re: [PATCH 4/4] define ASM_OUTPUT_LABEL to the name of a function

2015-08-05 Thread Richard Biener
On Wed, Aug 5, 2015 at 3:36 PM, Trevor Saunders wrote: > On Wed, Aug 05, 2015 at 01:47:30PM +0200, Richard Biener wrote: >> On Wed, Aug 5, 2015 at 12:57 PM, Trevor Saunders >> wrote: >> > On Mon, Jul 27, 2015 at 11:06:58AM +0200, Richard Biener wrote: >> >> On Sat, Jul 25, 2015 at 4:37 AM, wro

Re: [PATCH 4/4] define ASM_OUTPUT_LABEL to the name of a function

2015-08-05 Thread Trevor Saunders
On Wed, Aug 05, 2015 at 01:47:30PM +0200, Richard Biener wrote: > On Wed, Aug 5, 2015 at 12:57 PM, Trevor Saunders > wrote: > > On Mon, Jul 27, 2015 at 11:06:58AM +0200, Richard Biener wrote: > >> On Sat, Jul 25, 2015 at 4:37 AM, wrote: > >> > From: Trevor Saunders > >> > > >> > * conf

Re: [PATCH 4/4] define ASM_OUTPUT_LABEL to the name of a function

2015-08-05 Thread Richard Biener
On Wed, Aug 5, 2015 at 12:57 PM, Trevor Saunders wrote: > On Mon, Jul 27, 2015 at 11:06:58AM +0200, Richard Biener wrote: >> On Sat, Jul 25, 2015 at 4:37 AM, wrote: >> > From: Trevor Saunders >> > >> > * config/arc/arc.h, config/bfin/bfin.h, config/frv/frv.h, >> > config/ia64/ia

Re: [PATCH 4/4] define ASM_OUTPUT_LABEL to the name of a function

2015-08-05 Thread Trevor Saunders
On Mon, Jul 27, 2015 at 11:06:58AM +0200, Richard Biener wrote: > On Sat, Jul 25, 2015 at 4:37 AM, wrote: > > From: Trevor Saunders > > > > * config/arc/arc.h, config/bfin/bfin.h, config/frv/frv.h, > > config/ia64/ia64-protos.h, config/ia64/ia64.c, config/ia64/ia64.h, > >

Re: [PATCH 4/4] define ASM_OUTPUT_LABEL to the name of a function

2015-07-29 Thread Jeff Law
On 07/27/2015 09:47 PM, Trevor Saunders wrote: On Mon, Jul 27, 2015 at 11:06:58AM +0200, Richard Biener wrote: On Sat, Jul 25, 2015 at 4:37 AM, wrote: From: Trevor Saunders * config/arc/arc.h, config/bfin/bfin.h, config/frv/frv.h, config/ia64/ia64-protos.h, config/ia64/ia6

Re: [PATCH 4/4] define ASM_OUTPUT_LABEL to the name of a function

2015-07-27 Thread Trevor Saunders
On Mon, Jul 27, 2015 at 11:06:58AM +0200, Richard Biener wrote: > On Sat, Jul 25, 2015 at 4:37 AM, wrote: > > From: Trevor Saunders > > > > * config/arc/arc.h, config/bfin/bfin.h, config/frv/frv.h, > > config/ia64/ia64-protos.h, config/ia64/ia64.c, config/ia64/ia64.h, > >

Re: [PATCH 4/4] define ASM_OUTPUT_LABEL to the name of a function

2015-07-27 Thread Richard Biener
On Sat, Jul 25, 2015 at 4:37 AM, wrote: > From: Trevor Saunders > > * config/arc/arc.h, config/bfin/bfin.h, config/frv/frv.h, > config/ia64/ia64-protos.h, config/ia64/ia64.c, config/ia64/ia64.h, > config/lm32/lm32.h, config/mep/mep.h, config/mmix/mmix.h, > config/

Re: [PATCH 4/4] define ASM_OUTPUT_LABEL to the name of a function

2015-07-25 Thread Segher Boessenkool
On Sat, Jul 25, 2015 at 02:40:56PM -0400, Trevor Saunders wrote: > > If so many targets use default_assemble_label, can you make it an actual > > default instead of defining it everywhere? > > Well, it is the default, but many targets do over ride it to set it to > the default ;) Right, noticed

Re: [PATCH 4/4] define ASM_OUTPUT_LABEL to the name of a function

2015-07-25 Thread Trevor Saunders
On Sat, Jul 25, 2015 at 10:59:19AM -0500, Segher Boessenkool wrote: > On Fri, Jul 24, 2015 at 10:37:00PM -0400, tbsaunde+...@tbsaunde.org wrote: > > --- a/gcc/config/pa/pa.c > > +++ b/gcc/config/pa/pa.c > > @@ -9761,6 +9761,18 @@ pa_reloc_rw_mask (void) > >return 3; > > } > > > > +/* Assembl

Re: [PATCH 4/4] define ASM_OUTPUT_LABEL to the name of a function

2015-07-25 Thread Segher Boessenkool
On Fri, Jul 24, 2015 at 10:37:00PM -0400, tbsaunde+...@tbsaunde.org wrote: > --- a/gcc/config/pa/pa.c > +++ b/gcc/config/pa/pa.c > @@ -9761,6 +9761,18 @@ pa_reloc_rw_mask (void) >return 3; > } > > +/* Assemble a lable. */ Typo. > +void > +pa_output_label (FILE *f, const char *label) > +{

[PATCH 4/4] define ASM_OUTPUT_LABEL to the name of a function

2015-07-24 Thread tbsaunde+gcc
From: Trevor Saunders * config/arc/arc.h, config/bfin/bfin.h, config/frv/frv.h, config/ia64/ia64-protos.h, config/ia64/ia64.c, config/ia64/ia64.h, config/lm32/lm32.h, config/mep/mep.h, config/mmix/mmix.h, config/rs6000/rs6000.c, config/rs6000/xcoff.h, config/spu/sp