procyberian/gcc
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

tree-ssa-math-opts.c (bswap_stats): Add found_16bit field.

Browse source 
2012-09-26  Christophe Lyon <christophe.lyon@linaro.org>

	gcc/
	* tree-ssa-math-opts.c (bswap_stats): Add found_16bit field.
	(execute_optimize_bswap): Add support for builtin_bswap16.

	gcc/testsuite/
	* gcc.target/arm/builtin-bswap16-1.c: New testcase.

From-SVN: r191760
This commit is contained in:
Christophe Lyon 2012-09-26 13:42:47 +00:00 • committed by Christophe Lyon
parent eeca434ab6
commit 1df855ce35
4 changed files with 50 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
gcc/ChangeLog
View file

@ -1,3 +1,8 @@
2012-09-26 Christophe Lyon <christophe.lyon@linaro.org>
* tree-ssa-math-opts.c (bswap_stats): Add found_16bit field.
(execute_optimize_bswap): Add support for builtin_bswap16.
2012-09-26 Richard Guenther <rguenther@suse.de>
* tree.h (DECL_IS_BUILTIN): Compare LOCATION_LOCUS.

4
gcc/testsuite/ChangeLog
View file

@ -1,3 +1,7 @@
2012-09-26 Christophe Lyon <christophe.lyon@linaro.org>
* gcc.target/arm/builtin-bswap16-1.c: New testcase.
2012-09-25 Segher Boessenkool <segher@kernel.crashing.org>
PR target/51274

15
gcc/testsuite/gcc.target/arm/builtin-bswap16-1.c Normal file
View file

@ -0,0 +1,15 @@
/* { dg-do compile } */
/* { dg-options "-O2" } */
/* { dg-require-effective-target arm_arch_v6_ok } */
/* { dg-add-options arm_arch_v6 } */
/* { dg-final { scan-assembler-not "orr\[ \t\]" } } */
unsigned short swapu16_1 (unsigned short x)
{
return (x << 8) | (x >> 8);
}
unsigned short swapu16_2 (unsigned short x)
{
return (x >> 8) | (x << 8);
}

30
gcc/tree-ssa-math-opts.c
View file

@ -154,6 +154,9 @@ static struct
static struct
{
/* Number of hand-written 16-bit bswaps found. */
int found_16bit;
/* Number of hand-written 32-bit bswaps found. */
int found_32bit;
@ -1803,9 +1806,9 @@ static unsigned int
execute_optimize_bswap (void)
{
basic_block bb;
bool bswap32_p, bswap64_p;
bool bswap16_p, bswap32_p, bswap64_p;
bool changed = false;
tree bswap32_type = NULL_TREE, bswap64_type = NULL_TREE;
tree bswap16_type = NULL_TREE, bswap32_type = NULL_TREE, bswap64_type = NULL_TREE;
if (BITS_PER_UNIT != 8)
return 0;
@ -1813,17 +1816,25 @@ execute_optimize_bswap (void)
if (sizeof (HOST_WIDEST_INT) < 8)
return 0;
bswap16_p = (builtin_decl_explicit_p (BUILT_IN_BSWAP16)
&& optab_handler (bswap_optab, HImode) != CODE_FOR_nothing);
bswap32_p = (builtin_decl_explicit_p (BUILT_IN_BSWAP32)
&& optab_handler (bswap_optab, SImode) != CODE_FOR_nothing);
bswap64_p = (builtin_decl_explicit_p (BUILT_IN_BSWAP64)
&& (optab_handler (bswap_optab, DImode) != CODE_FOR_nothing
|| (bswap32_p && word_mode == SImode)));
if (!bswap32_p && !bswap64_p)
if (!bswap16_p && !bswap32_p && !bswap64_p)
return 0;
/* Determine the argument type of the builtins. The code later on
assumes that the return and argument type are the same. */
if (bswap16_p)
{
tree fndecl = builtin_decl_explicit (BUILT_IN_BSWAP16);
bswap16_type = TREE_VALUE (TYPE_ARG_TYPES (TREE_TYPE (fndecl)));
}
if (bswap32_p)
{
tree fndecl = builtin_decl_explicit (BUILT_IN_BSWAP32);
@ -1863,6 +1874,13 @@ execute_optimize_bswap (void)
switch (type_size)
{
case 16:
if (bswap16_p)
{
fndecl = builtin_decl_explicit (BUILT_IN_BSWAP16);
bswap_type = bswap16_type;
}
break;
case 32:
if (bswap32_p)
{
@ -1890,7 +1908,9 @@ execute_optimize_bswap (void)
continue;
changed = true;
if (type_size == 32)
if (type_size == 16)
bswap_stats.found_16bit++;
else if (type_size == 32)
bswap_stats.found_32bit++;
else
bswap_stats.found_64bit++;
@ -1935,6 +1955,8 @@ execute_optimize_bswap (void)
}
}
statistics_counter_event (cfun, "16-bit bswap implementations found",
bswap_stats.found_16bit);
statistics_counter_event (cfun, "32-bit bswap implementations found",
bswap_stats.found_32bit);
statistics_counter_event (cfun, "64-bit bswap implementations found",