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 tbsau...@tbsaunde.org 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

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

2015-08-07 Thread Richard Sandiford
Richard Biener richard.guent...@gmail.com 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 Sandiford
Trevor Saunders tbsau...@tbsaunde.org 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

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 tbsau...@tbsaunde.org wrote: On Fri, Aug 07, 2015 at 10:45:57AM +0100, Richard Sandiford wrote: Trevor Saunders tbsau...@tbsaunde.org writes: On Thu, Aug 06, 2015 at 08:36:36PM +0100, Richard Sandiford wrote: An integrated assembler or

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

2015-08-06 Thread Richard Sandiford
David Malcolm dmalc...@redhat.com 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

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 dmalc...@redhat.com 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:

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 Saunders

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, tbsaunde+...@tbsaunde.org wrote: From: Trevor Saunders tbsaunde+...@tbsaunde.org * config/arc/arc.h, config/bfin/bfin.h, config/frv/frv.h, config/ia64/ia64-protos.h,

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 tbsau...@tbsaunde.org wrote: On Mon, Jul 27, 2015 at 11:06:58AM +0200, Richard Biener wrote: On Sat, Jul 25, 2015 at 4:37 AM, tbsaunde+...@tbsaunde.org wrote: From: Trevor Saunders tbsaunde+...@tbsaunde.org * config/arc/arc.h,

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 tbsau...@tbsaunde.org wrote: On Mon, Jul 27, 2015 at 11:06:58AM +0200, Richard Biener wrote: On Sat, Jul 25, 2015 at 4:37 AM, tbsaunde+...@tbsaunde.org wrote: From: Trevor

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 tbsau...@tbsaunde.org 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 tbsau...@tbsaunde.org wrote: On Mon, Jul 27, 2015 at 11:06:58AM +0200, Richard Biener wrote: On Sat,

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 tbsau...@tbsaunde.org wrote: On Mon, Jul 27, 2015 at 11:06:58AM +0200, Richard Biener wrote: On Sat, Jul 25, 2015 at 4:37 AM, tbsaunde+...@tbsaunde.org wrote: From: Trevor Saunders

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 tbsau...@tbsaunde.org wrote: On Mon, Jul 27, 2015 at 11:06:58AM +0200, Richard Biener wrote: On Sat, Jul 25, 2015 at 4:37

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 tbsau...@tbsaunde.org wrote: On Mon, Jul 27, 2015 at

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_from_file

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, tbsaunde+...@tbsaunde.org wrote: From: Trevor Saunders tbsaunde+...@tbsaunde.org * config/arc/arc.h, config/bfin/bfin.h, config/frv/frv.h,

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, tbsaunde+...@tbsaunde.org wrote: From: Trevor Saunders tbsaunde+...@tbsaunde.org * config/arc/arc.h, config/bfin/bfin.h, config/frv/frv.h, config/ia64/ia64-protos.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, tbsaunde+...@tbsaunde.org wrote: From: Trevor Saunders tbsaunde+...@tbsaunde.org * 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,

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) +{ +

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; } +/* Assemble a lable. */

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 that

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

2015-07-24 Thread tbsaunde+gcc
From: Trevor Saunders tbsaunde+...@tbsaunde.org * 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,