Re: [PATCH, rs6000] Clean up capitalized diagnostic messages

2017-08-08 Thread Bill Schmidt
Fixed with r250952. Bill > On Aug 6, 2017, at 3:08 AM, Andreas Schwab wrote: > > FAIL: gcc.target/powerpc/bfp/scalar-extract-exp-2.c (test for errors, line > 18) > FAIL: gcc.target/powerpc/bfp/scalar-extract-exp-2.c (test for excess errors) > FAIL:

Re: [PATCH, rs6000] Clean up capitalized diagnostic messages

2017-08-07 Thread Segher Boessenkool
Hi Andreas, On Sun, Aug 06, 2017 at 10:08:23AM +0200, Andreas Schwab wrote: > FAIL: gcc.target/powerpc/bfp/scalar-extract-exp-2.c (test for errors, line > 18) > FAIL: gcc.target/powerpc/bfp/scalar-extract-exp-2.c (test for excess errors) > FAIL: gcc.target/powerpc/bfp/scalar-extract-exp-5.c

Re: [PATCH, rs6000] Clean up capitalized diagnostic messages

2017-08-06 Thread Andreas Schwab
FAIL: gcc.target/powerpc/bfp/scalar-extract-exp-2.c (test for errors, line 18) FAIL: gcc.target/powerpc/bfp/scalar-extract-exp-2.c (test for excess errors) FAIL: gcc.target/powerpc/bfp/scalar-extract-exp-5.c (test for errors, line 18) FAIL: gcc.target/powerpc/bfp/scalar-extract-exp-5.c (test for

Re: [PATCH, rs6000] Clean up capitalized diagnostic messages

2017-08-03 Thread Bill Schmidt
> On Aug 2, 2017, at 6:43 PM, Segher Boessenkool > wrote: > > Hi Bill, > > On Wed, Aug 02, 2017 at 10:29:20AM -0500, Bill Schmidt wrote: >> I don't anticipate backporting any of this. > > Good :-) > >> @@ -6802,7 +6802,7 @@ altivec_resolve_overloaded_builtin

Re: [PATCH, rs6000] Clean up capitalized diagnostic messages

2017-08-02 Thread Segher Boessenkool
Hi Bill, On Wed, Aug 02, 2017 at 10:29:20AM -0500, Bill Schmidt wrote: > I don't anticipate backporting any of this. Good :-) > @@ -6802,7 +6802,7 @@ altivec_resolve_overloaded_builtin (location_t loc > if (unsupported_builtin) >{ > const char *name =

Re: [PATCH, rs6000] Clean up capitalized diagnostic messages

2017-08-02 Thread Bill Schmidt
On Aug 2, 2017, at 11:04 AM, Joseph Myers wrote: > > On Wed, 2 Aug 2017, Jakub Jelinek wrote: > >> On Wed, Aug 02, 2017 at 10:29:20AM -0500, Bill Schmidt wrote: >>> --- gcc/config/rs6000/rs6000.c (revision 250791) >>> +++ gcc/config/rs6000/rs6000.c (working

Re: [PATCH, rs6000] Clean up capitalized diagnostic messages

2017-08-02 Thread Martin Sebor
On 08/02/2017 09:29 AM, Bill Schmidt wrote: Hi, Jakub pointed out that we've had several error messages slip into the POWER back end that incorrectly start with a capital letter. This patch hunts those down and cleans them up. Others have already commented on some aspects of the patch but

Re: [PATCH, rs6000] Clean up capitalized diagnostic messages

2017-08-02 Thread Joseph Myers
On Wed, 2 Aug 2017, Jakub Jelinek wrote: > On Wed, Aug 02, 2017 at 10:29:20AM -0500, Bill Schmidt wrote: > > --- gcc/config/rs6000/rs6000.c (revision 250791) > > +++ gcc/config/rs6000/rs6000.c (working copy) > > @@ -4132,7 +4132,7 @@ rs6000_option_override_internal (bool global_init_ >

Re: [PATCH, rs6000] Clean up capitalized diagnostic messages

2017-08-02 Thread Jakub Jelinek
On Wed, Aug 02, 2017 at 10:29:20AM -0500, Bill Schmidt wrote: > --- gcc/config/rs6000/rs6000.c(revision 250791) > +++ gcc/config/rs6000/rs6000.c(working copy) > @@ -4132,7 +4132,7 @@ rs6000_option_override_internal (bool global_init_ >|| rs6000_cpu == PROCESSOR_PPCE5500) >

[PATCH, rs6000] Clean up capitalized diagnostic messages

2017-08-02 Thread Bill Schmidt
Hi, Jakub pointed out that we've had several error messages slip into the POWER back end that incorrectly start with a capital letter. This patch hunts those down and cleans them up. Bootstrapped and tested on powerpc64le-linux-gnu with no regressions. Is this ok for trunk? I don't