mirror/gcc
2
0
Fork 0
mirror of git://gcc.gnu.org/git/gcc.git synced 2025-04-03 07:10:28 +08:00
Code Issues Packages Projects Releases Wiki Activity

rs6000: Use strlen instead of sizeof - 1

Browse Source 
It is easier to read and understand  strlen ("string")  than it is to
read and understand  sizeof ("string") - 1  .

	* config/rs6000/rs6000.c (rs6000_debug_print_mode): Don't use sizeof
	where strlen is more legible.
	(rs6000_builtin_vectorized_libmass): Ditto.
	(rs6000_print_options_internal): Ditto.
This commit is contained in:
Segher Boessenkool 2020-02-11 02:20:56 +00:00
parent 35e24106fc
commit d9e067f98b
2 changed files with 13 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
gcc/ChangeLog
View File

@ -1,3 +1,10 @@
2020-02-12 Segher Boessenkool <segher@kernel.crashing.org>
* config/rs6000/rs6000.c (rs6000_debug_print_mode): Don't use sizeof
where strlen is more legible.
(rs6000_builtin_vectorized_libmass): Ditto.
(rs6000_print_options_internal): Ditto.
2020-02-11 Martin Sebor <msebor@redhat.com>
PR tree-optimization/93683

12
gcc/config/rs6000/rs6000.c
View File

@ -2135,7 +2135,7 @@ rs6000_debug_print_mode (ssize_t m)
spaces = 0;
}
else
spaces += sizeof (" Reload=sl") - 1;
spaces += strlen (" Reload=sl");
if (reg_addr[m].scalar_in_vmx_p)
{
@ -2143,7 +2143,7 @@ rs6000_debug_print_mode (ssize_t m)
spaces = 0;
}
else
spaces += sizeof (" Upper=y") - 1;
spaces += strlen (" Upper=y");
if (rs6000_vector_unit[m] != VECTOR_NONE
|| rs6000_vector_mem[m] != VECTOR_NONE)
@ -5206,7 +5206,7 @@ rs6000_builtin_vectorized_libmass (combined_fn fn, tree type_out,
if (!bname)
return NULL_TREE;
strcpy (name, bname + sizeof ("__builtin_") - 1);
strcpy (name, bname + strlen ("__builtin_"));
strcat (name, suffix);
if (n_args == 1)
@ -23547,7 +23547,7 @@ rs6000_print_options_internal (FILE *file,
if ((flags & mask) == 0)
{
no_str = "no-";
len += sizeof ("no-") - 1;
len += strlen ("no-");
}
flags &= ~mask;
@ -23558,7 +23558,7 @@ rs6000_print_options_internal (FILE *file,
if ((flags & mask) != 0)
{
no_str = "no-";
len += sizeof ("no-") - 1;
len += strlen ("no-");
}
flags |= mask;
@ -23574,7 +23574,7 @@ rs6000_print_options_internal (FILE *file,
fprintf (file, "%s%s%s%s", comma, prefix, no_str, name);
comma = ", ";
comma_len = sizeof (", ") - 1;
comma_len = strlen (", ");
}
fputs ("\n", file);