fix diagnostic quoting/spelling in rs6000
authorMartin Liska <mliska@suse.cz>
Tue, 21 May 2019 06:55:56 +0000 (08:55 +0200)
committerMartin Liska <marxin@gcc.gnu.org>
Tue, 21 May 2019 06:55:56 +0000 (06:55 +0000)
2019-05-21  Martin Liska  <mliska@suse.cz>

* config/rs6000/driver-rs6000.c (elf_platform): Do not use
an extra newline.
* config/rs6000/rs6000-c.c (rs6000_pragma_longcal): Wrap pragma in %<%>.
(altivec_resolve_overloaded_builtin): Likewise for vec_lvsl and
vec_lvsr.
* config/rs6000/rs6000.c (rs6000_option_override_internal):
Quote a C type.
(rs6000_function_arg): Likewise.
(rs6000_expand_set_fpscr_drn_builtin): Remove trailing dot.
(rs6000_expand_ternop_builtin): Use interval syntax.
(get_element_number): Likewise.
(altivec_expand_builtin): Likewise.
(rs6000_get_function_versions_dispatcher): Quote target_clones.

Fix test-suite.
2019-05-21  Martin Liska  <mliska@suse.cz>

* gcc.target/powerpc/ppc64-abi-warn-1.c: Wrap a type.
* gcc.target/powerpc/pr80315-1.c: Use new interval format.
* gcc.target/powerpc/pr80315-2.c: Likewise.
* gcc.target/powerpc/pr80315-3.c: Likewise.
* gcc.target/powerpc/pr80315-4.c: Likewise.
* gcc.target/powerpc/warn-lvsl-lvsr.c: Wrap builtin names.

From-SVN: r271455

gcc/ChangeLog
gcc/config/rs6000/driver-rs6000.c
gcc/config/rs6000/rs6000-c.c
gcc/config/rs6000/rs6000.c
gcc/testsuite/ChangeLog
gcc/testsuite/gcc.target/powerpc/ppc64-abi-warn-1.c
gcc/testsuite/gcc.target/powerpc/pr80315-1.c
gcc/testsuite/gcc.target/powerpc/pr80315-2.c
gcc/testsuite/gcc.target/powerpc/pr80315-3.c
gcc/testsuite/gcc.target/powerpc/pr80315-4.c
gcc/testsuite/gcc.target/powerpc/warn-lvsl-lvsr.c

index 1c4e714e7d1057fe25579ef4c96845d102c44572..17b4b10aa824e8b8fc46d4812957751aa41c5266 100644 (file)
@@ -1,3 +1,21 @@
+2019-05-21  Martin Liska  <mliska@suse.cz>
+
+       * config/rs6000/driver-rs6000.c (elf_platform): Do not use
+       an extra newline.
+       * config/rs6000/rs6000-c.c (rs6000_pragma_longcal): Wrap pragma in %<%>.
+       (altivec_resolve_overloaded_builtin): Likewise for vec_lvsl and
+       vec_lvsr.
+       * config/rs6000/rs6000.c (rs6000_option_override_internal):
+       Quote a C type.
+       (rs6000_function_arg): Likewise.
+       (rs6000_expand_set_fpscr_drn_builtin): Remove trailing dot.
+       (rs6000_expand_ternop_builtin): Use interval syntax.
+       (get_element_number): Likewise.
+       (altivec_expand_builtin): Likewise.
+       (rs6000_get_function_versions_dispatcher): Quote target_clones.
+
+Fix test-suite.
+
 2019-05-20  Jakub Jelinek  <jakub@redhat.com>
 
        PR c++/59813
index b290eea4bdd373b61f8cb2d529039cd19f50f406..fde2a127ef1cf32162bf3cd0205b145a3d8d659a 100644 (file)
@@ -263,12 +263,10 @@ elf_platform (void)
          for (i = 0; i < ARRAY_SIZE (rs6000_supported_cpu_names); i++)
            candidates.safe_push (rs6000_supported_cpu_names[i]);
          candidates_list_and_hint (cpu, s, candidates);
-         fatal_error (
-           input_location,
-           "Unsupported cpu name returned from kernel for "
-           "%<-mcpu=native%>: %s\n"
-           "Please use an explicit cpu name.  Valid cpu names are: %s",
-           cpu, s);
+         error ("unsupported cpu name returned from kernel "
+                "for %<-mcpu=native%>: %s", cpu);
+         fatal_error (input_location, "please use an explicit cpu name; "
+                      "valid cpu names are: %s", s);
        }
     }
   return NULL;
index 7c28d4d81768892d8b8a6de2b77372049c30cde3..bd1e97e34dd9f64276fa499888b6b287e589d037 100644 (file)
    whether or not new function declarations receive a longcall
    attribute by default.  */
 
+void
+rs6000_pragma_longcall (cpp_reader *pfile ATTRIBUTE_UNUSED)
+{
 #define SYNTAX_ERROR(gmsgid) do {                                      \
   warning (OPT_Wpragmas, gmsgid);                                      \
-  warning (OPT_Wpragmas, "ignoring malformed #pragma longcall");       \
+  warning (OPT_Wpragmas, "ignoring malformed %<#pragma longcall%>");   \
   return;                                                              \
 } while (0)
 
-void
-rs6000_pragma_longcall (cpp_reader *pfile ATTRIBUTE_UNUSED)
-{
+
+
   tree x, n;
 
   /* If we get here, generic code has already scanned the directive
@@ -72,7 +74,7 @@ rs6000_pragma_longcall (cpp_reader *pfile ATTRIBUTE_UNUSED)
     SYNTAX_ERROR ("number must be 0 or 1");
 
   if (pragma_lex (&x) != CPP_EOF)
-    warning (OPT_Wpragmas, "junk at end of #pragma longcall");
+    warning (OPT_Wpragmas, "junk at end of %<#pragma longcall%>");
 
   rs6000_default_long_calls = (n == integer_one_node);
 }
@@ -6140,11 +6142,11 @@ altivec_resolve_overloaded_builtin (location_t loc, tree fndecl,
   /* vec_lvsl and vec_lvsr are deprecated for use with LE element order.  */
   if (fcode == ALTIVEC_BUILTIN_VEC_LVSL && !BYTES_BIG_ENDIAN)
     warning (OPT_Wdeprecated,
-            "vec_lvsl is deprecated for little endian; use "
+            "%<vec_lvsl%> is deprecated for little endian; use "
             "assignment for unaligned loads and stores");
   else if (fcode == ALTIVEC_BUILTIN_VEC_LVSR && !BYTES_BIG_ENDIAN)
     warning (OPT_Wdeprecated,
-            "vec_lvsr is deprecated for little endian; use "
+            "%<vec_lvsr%> is deprecated for little endian; use "
             "assignment for unaligned loads and stores");
 
   if (fcode == ALTIVEC_BUILTIN_VEC_MUL)
@@ -6846,7 +6848,7 @@ altivec_resolve_overloaded_builtin (location_t loc, tree fndecl,
        {
           if (TYPE_READONLY (TREE_TYPE (type))
              && !TYPE_READONLY (TREE_TYPE (decl_type)))
-           warning (0, "passing arg %d of %qE discards qualifiers from "
+           warning (0, "passing argument %d of %qE discards qualifiers from "
                        "pointer target type", n + 1, fndecl);
          type = build_pointer_type (build_qualified_type (TREE_TYPE (type),
                                                           0));
index b5dc5f30f8806e0c5720d5b228a263683fb2daf1..984298175393ac269254d830571bb9d5e6603b89 100644 (file)
@@ -4228,7 +4228,7 @@ rs6000_option_override_internal (bool global_init_p)
       if (main_target_opt != NULL
          && (main_target_opt->x_rs6000_long_double_type_size
              != default_long_double_size))
-       error ("target attribute or pragma changes long double size");
+       error ("target attribute or pragma changes %<long double%> size");
       else
        rs6000_long_double_type_size = default_long_double_size;
     }
@@ -4263,9 +4263,11 @@ rs6000_option_override_internal (bool global_init_p)
            {
              warned_change_long_double = true;
              if (TARGET_IEEEQUAD)
-               warning (OPT_Wpsabi, "Using IEEE extended precision long double");
+               warning (OPT_Wpsabi, "Using IEEE extended precision "
+                        "%<long double%>");
              else
-               warning (OPT_Wpsabi, "Using IBM extended precision long double");
+               warning (OPT_Wpsabi, "Using IBM extended precision "
+                        "%<long double%>");
            }
        }
     }
@@ -11744,7 +11746,7 @@ rs6000_function_arg (cumulative_args_t cum_v, machine_mode mode,
                {
                  warned = true;
                  inform (input_location,
-                         "the ABI of passing homogeneous float aggregates"
+                         "the ABI of passing homogeneous %<float%> aggregates"
                          " has changed in GCC 5");
                }
            }
@@ -13180,7 +13182,7 @@ rs6000_expand_set_fpscr_drn_builtin (enum insn_code icode, tree exp)
     /* Builtin not supported in 32-bit mode.  */
     fatal_error (input_location,
                 "%<__builtin_set_fpscr_drn%> is not supported "
-                "in 32-bit mode.");
+                "in 32-bit mode");
 
   if (rs6000_isa_flags & OPTION_MASK_SOFT_FLOAT)
     {
@@ -14203,7 +14205,7 @@ rs6000_expand_ternop_builtin (enum insn_code icode, tree exp, rtx target)
       if (TREE_CODE (arg2) != INTEGER_CST
          || wi::geu_p (wi::to_wide (arg2), 16))
        {
-         error ("argument 3 must be in the range 0..15");
+         error ("argument 3 must be in the range [0, 15]");
          return CONST0_RTX (tmode);
        }
     }
@@ -14336,7 +14338,7 @@ get_element_number (tree vec_type, tree arg)
   if (!tree_fits_uhwi_p (arg)
       || (elt = tree_to_uhwi (arg), elt > max))
     {
-      error ("selector must be an integer constant in the range 0..%wi", max);
+      error ("selector must be an integer constant in the range [0, %wi]", max);
       return 0;
     }
 
@@ -14656,7 +14658,7 @@ altivec_expand_builtin (tree exp, rtx target, bool *expandedp)
 
       if (TREE_CODE (arg1) != INTEGER_CST || TREE_INT_CST_LOW (arg1) > 12)
        {
-         error ("second argument to %qs must be 0..12", "vec_vextract4b");
+         error ("second argument to %qs must be [0, 12]", "vec_vextract4b");
          return expand_call (exp, target, false);
        }
       break;
@@ -14671,7 +14673,7 @@ altivec_expand_builtin (tree exp, rtx target, bool *expandedp)
 
       if (TREE_CODE (arg2) != INTEGER_CST || TREE_INT_CST_LOW (arg2) > 12)
        {
-         error ("third argument to %qs must be 0..12", "vec_vinsert4b");
+         error ("third argument to %qs must be [0, 12]", "vec_vinsert4b");
          return expand_call (exp, target, false);
        }
       break;
@@ -37203,7 +37205,7 @@ rs6000_get_function_versions_dispatcher (void *decl)
 
 #ifndef TARGET_LIBC_PROVIDES_HWCAP_IN_TCB
   error_at (DECL_SOURCE_LOCATION (default_node->decl),
-           "target_clones attribute needs GLIBC (2.23 and newer) that "
+           "%<target_clones%> attribute needs GLIBC (2.23 and newer) that "
            "exports hardware capability bits");
 #else
 
index bf9b4865502b10bd865120f806d5614bdf523459..4b8d6734a315993b216401c9af24c6b407bb5fa1 100644 (file)
@@ -1,3 +1,12 @@
+2019-05-21  Martin Liska  <mliska@suse.cz>
+
+       * gcc.target/powerpc/ppc64-abi-warn-1.c: Wrap a type.
+       * gcc.target/powerpc/pr80315-1.c: Use new interval format.
+       * gcc.target/powerpc/pr80315-2.c: Likewise.
+       * gcc.target/powerpc/pr80315-3.c: Likewise.
+       * gcc.target/powerpc/pr80315-4.c: Likewise.
+       * gcc.target/powerpc/warn-lvsl-lvsr.c: Wrap builtin names.
+
 2019-05-21  Martin Liska  <mliska@suse.cz>
 
        PR testsuite/90551
index c70c14c2858a191403d7a028743db67ca75f670b..ae7d5746b0d3ca0764bed277a6e5cbc28f66a679 100644 (file)
@@ -6,7 +6,7 @@ struct f8
     float x[8];
   };
 
-void test (struct f8 a, struct f8 b) /* { dg-message "note: the ABI of passing homogeneous float aggregates has changed" } */
+void test (struct f8 a, struct f8 b) /* { dg-message "note: the ABI of passing homogeneous 'float' aggregates has changed" } */
 {
 }
 
index d0fb6dc1c064a3d12a740d9b7f6a3155e100f93f..2f670a0c6c7c78753431f93cd2b487b20e8740cb 100644 (file)
@@ -10,6 +10,6 @@ main()
   int mask;
 
   /* Argument 2 must be 0 or 1.  Argument 3 must be in range 0..15.  */
-  res = __builtin_crypto_vshasigmaw (test, 1, 0xff); /* { dg-error "argument 3 must be in the range 0..15" } */
+  res = __builtin_crypto_vshasigmaw (test, 1, 0xff); /* { dg-error "argument 3 must be in the range \\[0, 15\\]" } */
   return 0;
 }
index 30df7f186583530a3078e6701d9a23d51ee919bd..e95187e21796dca7c6ac0f116e0f28ba713cfa0f 100644 (file)
@@ -10,6 +10,6 @@ main ()
   int mask;
 
   /* Argument 2 must be 0 or 1.  Argument 3 must be in range 0..15.  */
-  res = __builtin_crypto_vshasigmad (test, 1, 0xff); /* { dg-error "argument 3 must be in the range 0..15" } */
+  res = __builtin_crypto_vshasigmad (test, 1, 0xff); /* { dg-error "argument 3 must be in the range \\[0, 15\\]" } */
   return 0;
 }
index 5365f1d516c4a071d2e99688dbb07f8c8012fa8e..f1a782663a11131daa416c8c8f547d3b550410c9 100644 (file)
@@ -12,6 +12,6 @@ main ()
   int mask;
 
   /* Argument 2 must be 0 or 1.  Argument 3 must be in range 0..15.  */
-  res = vec_shasigma_be (test, 1, 0xff); /* { dg-error "argument 3 must be in the range 0..15" } */
+  res = vec_shasigma_be (test, 1, 0xff); /* { dg-error "argument 3 must be in the range \\[0, 15\\]" } */
   return res;
 }
index eb513639939149f20668e9dc6787e11a18292962..66a3efce9c520461a8f9e4e2dbd1628aa57f10bc 100644 (file)
@@ -12,6 +12,6 @@ main ()
   int mask;
 
   /* Argument 2 must be 0 or 1.  Argument 3 must be in range 0..15.  */
-  res = vec_shasigma_be (test, 1, 0xff); /* { dg-error "argument 3 must be in the range 0..15" } */
+  res = vec_shasigma_be (test, 1, 0xff); /* { dg-error "argument 3 must be in the range \\[0, 15\\]" } */
   return res;
 }
index bf889aaa22b5927f8e7536dc7979f7f2309cc918..62b37a9c3358f9f99c7c8d5d07034e988e0f054f 100644 (file)
@@ -9,6 +9,6 @@ float f[20];
 
 void foo ()
 {
-  vector unsigned char a = vec_lvsl (4, f); /* { dg-warning "vec_lvsl is deprecated for little endian; use assignment for unaligned loads and stores" } */
-  vector unsigned char b = vec_lvsr (8, f); /* { dg-warning "vec_lvsr is deprecated for little endian; use assignment for unaligned loads and stores" } */
+  vector unsigned char a = vec_lvsl (4, f); /* { dg-warning "'vec_lvsl' is deprecated for little endian; use assignment for unaligned loads and stores" } */
+  vector unsigned char b = vec_lvsr (8, f); /* { dg-warning "'vec_lvsr' is deprecated for little endian; use assignment for unaligned loads and stores" } */
 }