Hello.

The patch fixes -Wformat-diag in various targets.
I'm going to install the patch based on Jeff's pre-approval of similar patches.

Martin

gcc/ChangeLog:

        * collect2.cc (scan_libraries): Fix -Wformat-diag issues.
        * config/aarch64/aarch64-builtins.cc (aarch64_simd_expand_builtin): 
Likewise.
        * config/arc/arc.md: Likewise.
        * config/avr/avr.cc (avr_section_type_flags): Likewise.
        * config/bfin/bfin.cc (bfin_option_override): Likewise.
        (bfin_handle_longcall_attribute): Likewise.
        * config/cris/cris.h (FUNCTION_PROFILER): Likewise.
        * config/frv/frv.cc (frv_expand_builtin): Likewise.
        * config/ia64/ia64-c.cc (ia64_hpux_handle_builtin_pragma): Likewise.
        * config/iq2000/iq2000.cc (save_restore_insns): Likewise.
        (iq2000_print_operand_address): Likewise.
        (iq2000_print_operand): Likewise.
        * config/m32c/m32c-pragma.cc (m32c_pragma_memregs): Likewise.
        (m32c_pragma_address): Likewise.
        * config/m68k/m68k.cc (m68k_handle_fndecl_attribute): Likewise.
        * config/mips/mips.cc (mips_handle_interrupt_attr): Likewise.
        (mips_set_compression_mode): Likewise.
        * config/mmix/mmix.cc (mmix_function_profiler): Likewise.
        (mmix_print_operand): Likewise.
        (mmix_output_shiftvalue_op_from_str): Likewise.
        (mmix_output_shifted_value): Likewise.
        * config/msp430/driver-msp430.cc (msp430_select_hwmult_lib): Likewise.
        * config/msp430/msp430.cc (msp430_option_override): Likewise.
        (msp430_attr): Likewise.
        (msp430_expand_delay_cycles): Likewise.
        (msp430_expand_builtin): Likewise.
        * config/rs6000/aix73.h: Likewise.
        * config/rs6000/rtems.h (INVALID_64BIT): Likewise.
        * config/rx/rx.cc (rx_expand_builtin_mvtc): Likewise.
        (valid_psw_flag): Likewise.
        * config/sh/sh.cc (parse_validate_atomic_model_option): Likewise.
        * config/stormy16/stormy16.cc (xstormy16_function_profiler): Likewise.
        (xstormy16_expand_builtin_va_start): Likewise.
        (xstormy16_handle_below100_attribute): Likewise.
---
 gcc/collect2.cc                        |  2 +-
 gcc/config/aarch64/aarch64-builtins.cc |  2 +-
 gcc/config/arc/arc.md                  |  2 +-
 gcc/config/avr/avr.cc                  |  2 +-
 gcc/config/bfin/bfin.cc                |  8 +++----
 gcc/config/cris/cris.h                 |  2 +-
 gcc/config/frv/frv.cc                  |  6 +++---
 gcc/config/ia64/ia64-c.cc              |  2 +-
 gcc/config/iq2000/iq2000.cc            |  8 +++----
 gcc/config/m32c/m32c-pragma.cc         | 12 +++++------
 gcc/config/m68k/m68k.cc                |  2 +-
 gcc/config/mips/mips.cc                |  6 +++---
 gcc/config/mmix/mmix.cc                |  8 +++----
 gcc/config/msp430/driver-msp430.cc     |  6 +++---
 gcc/config/msp430/msp430.cc            | 30 +++++++++++++-------------
 gcc/config/rs6000/aix73.h              |  2 +-
 gcc/config/rs6000/rtems.h              |  2 +-
 gcc/config/rx/rx.cc                    |  4 ++--
 gcc/config/sh/sh.cc                    |  6 +++---
 gcc/config/stormy16/stormy16.cc        |  6 +++---
 20 files changed, 59 insertions(+), 59 deletions(-)

diff --git a/gcc/collect2.cc b/gcc/collect2.cc
index 0185b9d9e82..d81c7f28f16 100644
--- a/gcc/collect2.cc
+++ b/gcc/collect2.cc
@@ -2552,7 +2552,7 @@ scan_libraries (const char *prog_name)
pex = pex_init (PEX_USE_PIPES, "collect2", NULL);
   if (pex == NULL)
-    fatal_error (input_location, "pex_init failed: %m");
+    fatal_error (input_location, "%<pex_init%> failed: %m");
errmsg = pex_run (pex, 0, ldd_file_name, real_ldd_argv, NULL, NULL, &err);
   if (errmsg != NULL)
diff --git a/gcc/config/aarch64/aarch64-builtins.cc 
b/gcc/config/aarch64/aarch64-builtins.cc
index 215de1c09c8..b7f338d6229 100644
--- a/gcc/config/aarch64/aarch64-builtins.cc
+++ b/gcc/config/aarch64/aarch64-builtins.cc
@@ -1943,7 +1943,7 @@ aarch64_simd_expand_builtin (int fcode, tree exp, rtx 
target)
        }
       else
        error_at (EXPR_LOCATION (exp),
-                 "total size and element size must be a non-zero "
+                 "total size and element size must be a nonzero "
                  "constant immediate");
       /* Don't generate any RTL.  */
       return const0_rtx;
diff --git a/gcc/config/arc/arc.md b/gcc/config/arc/arc.md
index d4e22a4b2d9..ace3cb70424 100644
--- a/gcc/config/arc/arc.md
+++ b/gcc/config/arc/arc.md
@@ -4666,7 +4666,7 @@ (define_insn "trap_s"
   /* Keep this message in sync with the one in arc.cc:arc_expand_builtin,
      because *.md files do not get scanned by exgettext.  */
   fatal_error (input_location,
-              \"operand to trap_s should be an unsigned 6-bit value\");
+              \"operand to %<trap_s%> should be an unsigned 6-bit value\");
 }
   [(set_attr "length" "2")
   (set_attr "type" "misc")])
diff --git a/gcc/config/avr/avr.cc b/gcc/config/avr/avr.cc
index 9eec991e10a..4ed390e4cf9 100644
--- a/gcc/config/avr/avr.cc
+++ b/gcc/config/avr/avr.cc
@@ -10195,7 +10195,7 @@ avr_section_type_flags (tree decl, const char *name, 
int reloc)
        flags |= SECTION_BSS;  /* @nobits */
       else
        warning (0, "only uninitialized variables can be placed in the "
-                ".noinit section");
+                "%<.noinit%> section");
     }
if (decl && DECL_P (decl)
diff --git a/gcc/config/bfin/bfin.cc b/gcc/config/bfin/bfin.cc
index a89a1b48028..c749b4894d7 100644
--- a/gcc/config/bfin/bfin.cc
+++ b/gcc/config/bfin/bfin.cc
@@ -2382,10 +2382,10 @@ bfin_option_override (void)
     }
if (stack_limit_rtx && TARGET_STACK_CHECK_L1)
-    error ("can%'t use multiple stack checking methods together");
+    error ("cannot use multiple stack checking methods together");
if (TARGET_ID_SHARED_LIBRARY && TARGET_FDPIC)
-    error ("ID shared libraries and FD-PIC mode can%'t be used together");
+    error ("ID shared libraries and FD-PIC mode cannot be used together");
/* Don't allow the user to specify -mid-shared-library and -msep-data
      together, as it makes little sense from a user's point of view...  */
@@ -2419,7 +2419,7 @@ bfin_option_override (void)
     error ("%<-mcoreb%> should be used with %<-mmulticore%>");
if (TARGET_COREA && TARGET_COREB)
-    error ("%<-mcorea%> and %<-mcoreb%> can%'t be used together");
+    error ("%<-mcorea%> and %<-mcoreb%> cannot be used together");
flag_schedule_insns = 0; @@ -4763,7 +4763,7 @@ bfin_handle_longcall_attribute (tree *node, tree name,
          && lookup_attribute ("longcall", TYPE_ATTRIBUTES (*node))))
     {
       warning (OPT_Wattributes,
-              "can%'t apply both longcall and shortcall attributes to the same 
function");
+              "cannott apply both longcall and shortcall attributes to the same 
function");
       *no_add_attrs = true;
     }
diff --git a/gcc/config/cris/cris.h b/gcc/config/cris/cris.h
index c623d083ff1..b274e116620 100644
--- a/gcc/config/cris/cris.h
+++ b/gcc/config/cris/cris.h
@@ -663,7 +663,7 @@ struct cum_args {int regs;};
 /* Node: Profiling */
#define FUNCTION_PROFILER(FILE, LABELNO) \
- error ("no FUNCTION_PROFILER for CRIS")
+ error ("no %<FUNCTION_PROFILER%> for CRIS")
/* FIXME: Some of the undefined macros might be mandatory. If so, fix
    documentation.  */
diff --git a/gcc/config/frv/frv.cc b/gcc/config/frv/frv.cc
index d6c399304a1..6f1904b358c 100644
--- a/gcc/config/frv/frv.cc
+++ b/gcc/config/frv/frv.cc
@@ -9134,7 +9134,7 @@ frv_expand_builtin (tree exp,
     case FRV_BUILTIN_IACCsetl:
       if (!TARGET_FR405_BUILTINS)
        {
-         error ("this builtin function is only available"
+         error ("this built-in function is only available"
                 " on the fr405 and fr450");
          return NULL_RTX;
        }
@@ -9143,7 +9143,7 @@ frv_expand_builtin (tree exp,
     case FRV_BUILTIN_PREFETCH:
       if (!TARGET_FR500_FR550_BUILTINS)
        {
-         error ("this builtin function is only available on the fr500"
+         error ("this built-in function is only available on the fr500"
                 " and fr550");
          return NULL_RTX;
        }
@@ -9155,7 +9155,7 @@ frv_expand_builtin (tree exp,
     case FRV_BUILTIN_MQSRAHI:
       if (!TARGET_MEDIA_FR450)
        {
-         error ("this builtin function is only available on the fr450");
+         error ("this built-in function is only available on the fr450");
          return NULL_RTX;
        }
       break;
diff --git a/gcc/config/ia64/ia64-c.cc b/gcc/config/ia64/ia64-c.cc
index f1c8948fa30..fa1b3817d93 100644
--- a/gcc/config/ia64/ia64-c.cc
+++ b/gcc/config/ia64/ia64-c.cc
@@ -47,7 +47,7 @@ ia64_hpux_handle_builtin_pragma (cpp_reader *pfile 
ATTRIBUTE_UNUSED)
        type = pragma_lex (&x);
     }
   if (type != CPP_EOF)
-    warning (OPT_Wpragmas, "malformed #pragma builtin");
+    warning (OPT_Wpragmas, "malformed %<#pragma builtin%>");
 }
/* List of standard math functions which do not set matherr by default
diff --git a/gcc/config/iq2000/iq2000.cc b/gcc/config/iq2000/iq2000.cc
index 9b8349d7a88..062734102ba 100644
--- a/gcc/config/iq2000/iq2000.cc
+++ b/gcc/config/iq2000/iq2000.cc
@@ -1864,7 +1864,7 @@ save_restore_insns (int store_p)
if (gp_offset < 0 || end_offset < 0)
     internal_error
-      ("gp_offset (%ld) or end_offset (%ld) is less than zero",
+      ("%<gp_offset%> (%ld) or %<end_offset%> (%ld) is less than zero",
        (long) gp_offset, (long) end_offset);
else if (gp_offset < 32768)
@@ -2890,7 +2890,7 @@ static void
 iq2000_print_operand_address (FILE * file, machine_mode mode, rtx addr)
 {
   if (!addr)
-    error ("PRINT_OPERAND_ADDRESS, null pointer");
+    error ("%<PRINT_OPERAND_ADDRESS%>, null pointer");
else
     switch (GET_CODE (addr))
@@ -3045,7 +3045,7 @@ iq2000_print_operand (FILE *file, rtx op, int letter)
          break;
default:
-         error ("PRINT_OPERAND: Unknown punctuation %<%c%>", letter);
+         error ("%<PRINT_OPERAND%>: Unknown punctuation %<%c%>", letter);
          break;
        }
@@ -3054,7 +3054,7 @@ iq2000_print_operand (FILE *file, rtx op, int letter) if (! op)
     {
-      error ("PRINT_OPERAND null pointer");
+      error ("%<PRINT_OPERAND%> null pointer");
       return;
     }
diff --git a/gcc/config/m32c/m32c-pragma.cc b/gcc/config/m32c/m32c-pragma.cc
index 685caeaceac..c7d01a99d14 100644
--- a/gcc/config/m32c/m32c-pragma.cc
+++ b/gcc/config/m32c/m32c-pragma.cc
@@ -51,14 +51,14 @@ m32c_pragma_memregs (cpp_reader * reader ATTRIBUTE_UNUSED)
type = pragma_lex (&val);
          if (type != CPP_EOF)
-           warning (0, "junk at end of #pragma GCC memregs [0..16]");
+           warning (0, "junk at end of %<#pragma GCC memregs%> [0-16]");
if (i >= 0 && i <= 16)
            {
              if (!ok_to_change_target_memregs)
                {
                  warning (0,
-                          "#pragma GCC memregs must precede any function 
decls");
+                          "%<#pragma GCC memregs%> must precede any function 
declarations");
                  return;
                }
              target_memregs = i;
@@ -66,14 +66,14 @@ m32c_pragma_memregs (cpp_reader * reader ATTRIBUTE_UNUSED)
            }
          else
            {
-             warning (0, "#pragma GCC memregs takes a number [0..16]");
+             warning (0, "%<#pragma GCC memregs%> takes a number [0-16]");
            }
return;
        }
     }
- error ("#pragma GCC memregs takes a number [0..16]");
+  error ("%<#pragma GCC memregs%> takes a number [0-16]");
 }
/* Implements the "pragma ADDRESS" pragma. This pragma takes a
@@ -101,12 +101,12 @@ m32c_pragma_address (cpp_reader * reader ATTRIBUTE_UNUSED)
          type = pragma_lex (&var);
          if (type != CPP_EOF)
            {
-             error ("junk at end of #pragma ADDRESS");
+             error ("junk at end of %<#pragma ADDRESS%>");
            }
          return;
        }
     }
-  error ("malformed #pragma ADDRESS variable address");
+  error ("malformed %<#pragma ADDRESS%> variable address");
 }
/* Implements REGISTER_TARGET_PRAGMAS. */
diff --git a/gcc/config/m68k/m68k.cc b/gcc/config/m68k/m68k.cc
index 1d5ef02f8de..62898dafe92 100644
--- a/gcc/config/m68k/m68k.cc
+++ b/gcc/config/m68k/m68k.cc
@@ -803,7 +803,7 @@ m68k_handle_fndecl_attribute (tree *node, tree name,
   if (!TARGET_FIDOA
       && !strcmp (IDENTIFIER_POINTER (name), "interrupt_thread"))
     {
-      error ("interrupt_thread is available only on fido");
+      error ("%<interrupt_thread%> is available only on fido");
       *no_add_attrs = true;
     }
diff --git a/gcc/config/mips/mips.cc b/gcc/config/mips/mips.cc
index 6c0da2bb90c..4f9683e8bf4 100644
--- a/gcc/config/mips/mips.cc
+++ b/gcc/config/mips/mips.cc
@@ -1519,7 +1519,7 @@ mips_handle_interrupt_attr (tree *node ATTRIBUTE_UNUSED, 
tree name, tree args,
        {
          warning (OPT_Wattributes,
                   "argument to %qE attribute is neither eic, nor "
-                  "vector=<line>", name);
+                  "%<vector=<line>%>", name);
          *no_add_attrs = true;
        }
       else if (startswith (TREE_STRING_POINTER (cst), "vector="))
@@ -1535,7 +1535,7 @@ mips_handle_interrupt_attr (tree *node ATTRIBUTE_UNUSED, 
tree name, tree args,
            {
              warning (OPT_Wattributes,
                       "interrupt vector to %qE attribute is not "
-                      "vector=(sw0|sw1|hw0|hw1|hw2|hw3|hw4|hw5)",
+                      "%<vector=(sw0|sw1|hw0|hw1|hw2|hw3|hw4|hw5)%>",
                       name);
              *no_add_attrs = true;
            }
@@ -19712,7 +19712,7 @@ mips_set_compression_mode (unsigned int 
compression_mode)
        sorry ("MIPS16 %<-mxgot%> code");
if (TARGET_HARD_FLOAT_ABI && !TARGET_OLDABI)
-       sorry ("hard-float MIPS16 code for ABIs other than o32 and o64");
+       sorry ("%<hard-float%> MIPS16 code for ABIs other than o32 and o64");
if (TARGET_MSA)
        sorry ("MSA MIPS16 code");
diff --git a/gcc/config/mmix/mmix.cc b/gcc/config/mmix/mmix.cc
index 18053d453be..439ae8f044f 100644
--- a/gcc/config/mmix/mmix.cc
+++ b/gcc/config/mmix/mmix.cc
@@ -977,7 +977,7 @@ void
 mmix_function_profiler (FILE *stream ATTRIBUTE_UNUSED,
                        int labelno ATTRIBUTE_UNUSED)
 {
-  sorry ("function_profiler support for MMIX");
+  sorry ("%<function_profiler%> support for MMIX");
 }
/* Worker function for TARGET_SETUP_INCOMING_VARARGS. For the moment,
@@ -1763,7 +1763,7 @@ mmix_print_operand (FILE *stream, rtx x, int code)
default:
       /* Presumably there's a missing case above if we get here.  */
-      internal_error ("MMIX Internal: Missing %qc case in mmix_print_operand", 
code);
+      internal_error ("MMIX Internal: Missing %qc case in 
%<mmix_print_operand%>", code);
     }
switch (GET_CODE (modified_x))
@@ -2591,7 +2591,7 @@ mmix_output_shiftvalue_op_from_str (FILE *stream,
     {
       char s[sizeof ("0xffffffffffffffff")];
       sprintf (s, "%#" PRIx64, value);
-      internal_error ("MMIX Internal: %s is not a shiftable int", s);
+      internal_error ("MMIX Internal: %s is not a shiftable integer", s);
     }
for (i = 0; i < 4; i++)
@@ -2650,7 +2650,7 @@ mmix_output_shifted_value (FILE *stream, int64_t value)
     {
       char s[16+2+1];
       sprintf (s, "%#" PRIx64, value);
-      internal_error ("MMIX Internal: %s is not a shiftable int", s);
+      internal_error ("MMIX Internal: %s is not a shiftable integer", s);
     }
for (i = 0; i < 4; i++)
diff --git a/gcc/config/msp430/driver-msp430.cc 
b/gcc/config/msp430/driver-msp430.cc
index 5ea2b078fb5..b4bff424f55 100644
--- a/gcc/config/msp430/driver-msp430.cc
+++ b/gcc/config/msp430/driver-msp430.cc
@@ -89,7 +89,7 @@ msp430_select_hwmult_lib (int argc ATTRIBUTE_UNUSED,
     {
     case 1:
       if (strcasecmp (argv[0], "default"))
-       error ("unexpected argument to msp430_select_hwmult_lib: %s", argv[0]);
+       error ("unexpected argument to %<msp430_select_hwmult_lib%>: %s", 
argv[0]);
       break;
default:
@@ -138,12 +138,12 @@ msp430_select_hwmult_lib (int argc ATTRIBUTE_UNUSED,
            }
        }
       else
-       error ("unexpected first argument to msp430_select_hwmult_lib: %s",
+       error ("unexpected first argument to %<msp430_select_hwmult_lib%>: %s",
               argv[0]);
       break;
case 0:
-      error ("msp430_select_hwmult_lib needs one or more arguments");
+      error ("%<msp430_select_hwmult_lib%> needs one or more arguments");
       break;
     }
diff --git a/gcc/config/msp430/msp430.cc b/gcc/config/msp430/msp430.cc
index 18673ff6fd3..eb219fd5e71 100644
--- a/gcc/config/msp430/msp430.cc
+++ b/gcc/config/msp430/msp430.cc
@@ -222,16 +222,16 @@ msp430_option_override (void)
                {
                  if (target_cpu == MSP430_CPU_MSP430X_DEFAULT)
                    warning (0,
-                            "Unrecognized MCU name %qs, assuming that it is "
-                            "just a MSP430X with no hardware multiply.\n"
-                            "Use the %<-mcpu%> and %<-mhwmult%> options to "
-                            "set these explicitly.",
+                            "unrecognized MCU name %qs, assuming that it is "
+                            "just a MSP430X with no hardware multiply; "
+                            "use the %<-mcpu%> and %<-mhwmult%> options to "
+                            "set these explicitly",
                             target_mcu);
                  else
                    warning (0,
-                            "Unrecognized MCU name %qs, assuming that it "
-                            "has no hardware multiply.\nUse the %<-mhwmult%> "
-                            "option to set this explicitly.",
+                            "unrecognized MCU name %qs, assuming that it "
+                            "has no hardware multiply; use the %<-mhwmult%> "
+                            "option to set this explicitly",
                             target_mcu);
                }
@@ -241,9 +241,9 @@ msp430_option_override (void)
            {
              if (msp430_warn_mcu)
                warning (0,
-                        "Unrecognized MCU name %qs, assuming that it just "
-                        "supports the MSP430X ISA.\nUse the %<-mcpu%> option "
-                        "to set the ISA explicitly.",
+                        "unrecognized MCU name %qs, assuming that it just "
+                        "supports the MSP430X ISA; use the %<-mcpu%> option "
+                        "to set the ISA explicitly",
                         target_mcu);
            }
          else if (msp430_warn_mcu)
@@ -1901,7 +1901,7 @@ msp430_attr (tree * node,
            /* Allow the attribute to be added - the linker script
               being used may still recognise this value.  */
            warning (OPT_Wattributes,
-                    "numeric argument of %qE attribute must be in range 0..63",
+                    "numeric argument of %qE attribute must be in range 
[0-63]",
                     name);
          break;
@@ -2744,7 +2744,7 @@ msp430_expand_delay_cycles (rtx arg) if (GET_CODE (arg) != CONST_INT)
     {
-      error ("__delay_cycles() only takes constant arguments");
+      error ("%<__delay_cycles()%> only takes constant arguments");
       return NULL_RTX;
     }
@@ -2754,7 +2754,7 @@ msp430_expand_delay_cycles (rtx arg)
     {
       if (c < 0)
        {
-         error ("__delay_cycles only takes non-negative cycle counts");
+         error ("%<__delay_cycles%> only takes non-negative cycle counts");
          return NULL_RTX;
        }
     }
@@ -2775,7 +2775,7 @@ msp430_expand_delay_cycles (rtx arg)
          c %= 4;
          if ((unsigned long long) i > 0xffffffffULL)
            {
-             error ("__delay_cycles is limited to 32-bit loop counts");
+             error ("%<__delay_cycles%> is limited to 32-bit loop counts");
              return NULL_RTX;
            }
        }
@@ -2846,7 +2846,7 @@ msp430_expand_builtin (tree exp,
if (! msp430_is_interrupt_func ())
     {
-      error ("MSP430 builtin functions only work inside interrupt handlers");
+      error ("MSP430 built-in functions only work inside interrupt handlers");
       return NULL_RTX;
     }
diff --git a/gcc/config/rs6000/aix73.h b/gcc/config/rs6000/aix73.h
index 8baa27e54c0..0b847900d3a 100644
--- a/gcc/config/rs6000/aix73.h
+++ b/gcc/config/rs6000/aix73.h
@@ -34,7 +34,7 @@ do {                                                          
        \
     {                                                                  \
       rs6000_long_double_type_size = 64;                               \
       if (OPTION_SET_P (rs6000_long_double_type_size))         \
-       warning (0, "soft-float and long-double-128 are incompatible");       \
+       warning (0, "%<soft-float%> and long-double-128 are incompatible");     
\
     }                                                                  \
   if (TARGET_POWERPC64 && ! TARGET_64BIT)                              \
     {                                                                  \
diff --git a/gcc/config/rs6000/rtems.h b/gcc/config/rs6000/rtems.h
index a6520aa363d..8aa41d5448d 100644
--- a/gcc/config/rs6000/rtems.h
+++ b/gcc/config/rs6000/rtems.h
@@ -82,7 +82,7 @@
 #define        RS6000_ABI_NAME "linux"
/* Copy and paste from linux64.h and freebsd64.h */
-#define INVALID_64BIT "-m%s not supported in this configuration"
+#define INVALID_64BIT "%<-m%s%> not supported in this configuration"
/* A lot of copy and paste from linux64.h and freebsd64.h */
 #undef SUBSUBTARGET_OVERRIDE_OPTIONS
diff --git a/gcc/config/rx/rx.cc b/gcc/config/rx/rx.cc
index ba82482b664..7c3975e63dd 100644
--- a/gcc/config/rx/rx.cc
+++ b/gcc/config/rx/rx.cc
@@ -2477,7 +2477,7 @@ rx_expand_builtin_mvtc (tree exp)
if (INTVAL (arg1) == 1)
     {
-      warning (0, "invalid control register for mvtc : %d - using 'psw'",
+      warning (0, "invalid control register for mvtc: %d - using 'psw'",
               (int) INTVAL (arg1));
       arg1 = const0_rtx;
     }
@@ -2602,7 +2602,7 @@ valid_psw_flag (rtx op, const char *which)
   error ("%<__builtin_rx_%s%> takes %<C%>, %<Z%>, %<S%>, %<O%>, %<I%>, "
         "or %<U%>", which);
   if (!mvtc_inform_done)
-    error ("use %<__builtin_rx_mvtc%> (0, ... ) to write arbitrary values to 
PSW");
+    error ("use %<__builtin_rx_mvtc (0, ... )%> to write arbitrary values to 
PSW");
   mvtc_inform_done = 1;
return 0;
diff --git a/gcc/config/sh/sh.cc b/gcc/config/sh/sh.cc
index dd8bce4aa3b..8d4056338a5 100644
--- a/gcc/config/sh/sh.cc
+++ b/gcc/config/sh/sh.cc
@@ -728,7 +728,7 @@ parse_validate_atomic_model_option (const char* str)
          goto got_mode_name;
        }
- err_ret ("invalid atomic model name \"%s\"", tokens.front ().c_str ());
+    err_ret ("invalid atomic model name %qs", tokens.front ().c_str ());
 got_mode_name:;
   }
@@ -742,11 +742,11 @@ got_mode_name:;
          std::string offset_str = tokens[i].substr (strlen ("gbr-offset="));
          ret.tcb_gbr_offset = integral_argument (offset_str.c_str ());
          if (offset_str.empty () || ret.tcb_gbr_offset == -1)
-           err_ret ("could not parse gbr-offset value \"%s\" in atomic model "
+           err_ret ("could not parse gbr-offset value %qs in atomic model "
                     "option", offset_str.c_str ());
        }
       else
-       err_ret ("unknown parameter \"%s\" in atomic model option",
+       err_ret ("unknown parameter %qs in atomic model option",
                 tokens[i].c_str ());
     }
diff --git a/gcc/config/stormy16/stormy16.cc b/gcc/config/stormy16/stormy16.cc
index 36fb97eb2e5..fabf09ab9a0 100644
--- a/gcc/config/stormy16/stormy16.cc
+++ b/gcc/config/stormy16/stormy16.cc
@@ -1212,7 +1212,7 @@ xstormy16_epilogue_uses (int regno)
 void
 xstormy16_function_profiler (void)
 {
-  sorry ("function_profiler support");
+  sorry ("%<function_profiler%> support");
 }
 
 /* Update CUM to advance past argument ARG.  Once this is done,
@@ -1306,7 +1306,7 @@ xstormy16_expand_builtin_va_start (tree valist, rtx 
nextarg ATTRIBUTE_UNUSED)
   tree t,u;
if (xstormy16_interrupt_function_p ())
-    error ("cannot use va_start in interrupt function");
+    error ("cannot use %<va_start%> in interrupt function");
f_base = TYPE_FIELDS (va_list_type_node);
   f_count = DECL_CHAIN (f_base);
@@ -2256,7 +2256,7 @@ xstormy16_handle_below100_attribute (tree *node,
     {
       if (! (TREE_PUBLIC (*node) || TREE_STATIC (*node)))
        {
-         warning (OPT_Wattributes, "__BELOW100__ attribute not allowed "
+         warning (OPT_Wattributes, "%<__BELOW100__%> attribute not allowed "
                   "with auto storage class");
          *no_add_attrs = true;
        }
--
2.34.1

Reply via email to