xtensa: make map_suffix_reloc_to_operator return operatorT

It always returns an element of the enum operatorT, so it should be clearer to
make that the return type.

gas/ChangeLog:

2016-05-24  Trevor Saunders  <tbsaunde+binutils@tbsaunde.org>

	* config/tc-xtensa.c (struct suffix_reloc_map): Change type of field
	operator to operatorT.
	(map_suffix_reloc_to_operator): Change return type to operatorT.
This commit is contained in:
Trevor Saunders 2016-05-23 00:39:47 -04:00
parent c023823f5f
commit cc34adb290
2 changed files with 10 additions and 4 deletions

View File

@ -1,3 +1,9 @@
2016-05-24 Trevor Saunders <tbsaunde+binutils@tbsaunde.org>
* config/tc-xtensa.c (struct suffix_reloc_map): Change type of field
operator to operatorT.
(map_suffix_reloc_to_operator): Change return type to operatorT.
2016-05-24 Trevor Saunders <tbsaunde+binutils@tbsaunde.org>
* config/tc-d30v.c (find_format): Change type of X_op to operatorT.

View File

@ -364,7 +364,7 @@ struct suffix_reloc_map
const char *suffix;
int length;
bfd_reloc_code_real_type reloc;
unsigned char operator;
operatorT operator;
};
#define SUFFIX_MAP(str, reloc, op) { str, sizeof (str) - 1, reloc, op }
@ -1773,11 +1773,11 @@ xtensa_elf_suffix (char **str_p, expressionS *exp_p)
/* Find the matching operator type. */
static unsigned char
static operatorT
map_suffix_reloc_to_operator (bfd_reloc_code_real_type reloc)
{
struct suffix_reloc_map *sfx;
unsigned char operator = (unsigned char) -1;
operatorT operator = O_illegal;
for (sfx = &suffix_relocs[0]; sfx->suffix; sfx++)
{
@ -1787,7 +1787,7 @@ map_suffix_reloc_to_operator (bfd_reloc_code_real_type reloc)
break;
}
}
gas_assert (operator != (unsigned char) -1);
gas_assert (operator != O_illegal);
return operator;
}