public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [committed] mips: Fix up -Wliteral-suffix warning on mti-linux.h
@ 2020-04-02 13:29 Jakub Jelinek
  0 siblings, 0 replies; only message in thread
From: Jakub Jelinek @ 2020-04-02 13:29 UTC (permalink / raw)
  To: gcc-patches

Hi!

I've noticed while trying to reproduce PR92989 the following warning:
In file included from ./tm.h:42,
                 from ../../gcc/backend.h:28,
                 from ../../gcc/lra-assigns.c:80:
../../gcc/config/mips/mti-linux.h:31:5: warning: invalid suffix on literal; C++11 requires a space between literal and string macro [-Wliteral-suffix]
 "/%{mmicromips:micro}mips%{mel|EL:el}-"MIPS_SYSVERSION_SPEC  \
 ^
This fixes it, string concatenation works just fine even with whitespace
in between.

Committed to trunk as obvious.

2020-04-02  Jakub Jelinek  <jakub@redhat.com>

	* config/mips/mti-linux.h (SYSROOT_SUFFIX_SPEC): Add a space in
	between a string literal and MIPS_SYSVERSION_SPEC macro.

--- gcc/config/mips/mti-linux.h.jj	2020-01-12 11:54:36.360414208 +0100
+++ gcc/config/mips/mti-linux.h	2020-04-02 15:22:32.478597778 +0200
@@ -28,7 +28,7 @@ along with GCC; see the file COPYING3.
 
 #undef SYSROOT_SUFFIX_SPEC
 #define SYSROOT_SUFFIX_SPEC						\
-    "/%{mmicromips:micro}mips%{mel|EL:el}-"MIPS_SYSVERSION_SPEC		\
+    "/%{mmicromips:micro}mips%{mel|EL:el}-" MIPS_SYSVERSION_SPEC	\
     "%{msoft-float:-soft;:-hard}"					\
     "%{!mips32r6:%{!mips64r6:%{mnan=2008:-nan2008}}}%{muclibc:-uclibc}"
 


	Jakub


^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2020-04-02 13:30 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-04-02 13:29 [committed] mips: Fix up -Wliteral-suffix warning on mti-linux.h Jakub Jelinek

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).