This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [PATCH, rs6000] More diagnostic cleanup, addressing PR79845

From: Martin Sebor <msebor at gmail dot com>
To: Bill Schmidt <wschmidt at linux dot vnet dot ibm dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>
Cc: Segher Boessenkool <segher at kernel dot crashing dot org>, David Edelsohn <dje dot gcc at gmail dot com>
Date: Sun, 13 Aug 2017 14:41:26 -0600
Subject: Re: [PATCH, rs6000] More diagnostic cleanup, addressing PR79845
Authentication-results: sourceware.org; auth=none
References: <e6da7858-91d3-8b4b-8686-b6591dbd902f@linux.vnet.ibm.com>

--- gcc/config/rs6000/rs6000-c.c	(revision 250965)
+++ gcc/config/rs6000/rs6000-c.c	(working copy)
@@ -5812,7 +5812,7 @@ altivec_resolve_overloaded_builtin (location_t loc
 	 types.  */
       if (nargs != 2)
 	{
-	  error ("vec_mul only accepts 2 arguments");
+	  error ("builtin %qs only accepts 2 arguments", "vec_mul");


I don' meant to nit-pick too much but it has been drilled into me
that the term is built-in, not builtin.  There are a number of
deviations from this preferred spelling in GCC messages already
(mainly though not exclusively in the rs600 back-end) but I think
it would be nice to converge on the preferred spelling.

https://gcc.gnu.org/codingconventions.html#Spelling

Martin

Follow-Ups:
- Re: [PATCH, rs6000] More diagnostic cleanup, addressing PR79845
  - From: Bill Schmidt

References:
- [PATCH, rs6000] More diagnostic cleanup, addressing PR79845
  - From: Bill Schmidt

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]