mirrors/binutils-gdb
0
0
Fork 0
mirror of https://sourceware.org/git/binutils-gdb.git synced 2024-11-30 05:23:49 +08:00
Code Issues Actions Packages Projects Releases Wiki Activity

x86: remove pointless 2nd parameter from check_VecOperations()

Browse Source 
In the one case where non-NULL gets passed, passing NULL has the same
effect. Hence the parameter is not needed at all.
This commit is contained in:
Jan Beulich 2021-06-07 12:01:14 +02:00
parent 4e014f6ac0
commit f70c6814bb
3 changed files with 11 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
gas/ChangeLog
View File

@ -1,3 +1,9 @@
2021-06-07 Jan Beulich <jbeulich@suse.com>
* config/tc-i386.c (check_VecOperations): Remove 2nd parameter.
(i386_att_operand): Adjust calls to check_VecOperations.
* config/tc-i386-intel.c (i386_intel_operand): Likewise.
2021-06-07 Jan Beulich <jbeulich@suse.com>
* config/tc-i386.c (i386_immediate): Don't call check_VecOperations.

2
gas/config/tc-i386-intel.c
View File

@ -606,7 +606,7 @@ i386_intel_operand (char *operand_string, int got_a_float)
/* Handle vector operations. */
if (*input_line_pointer == '{')
{
char *end = check_VecOperations (input_line_pointer, NULL);
char *end = check_VecOperations (input_line_pointer);
if (end)
input_line_pointer = end;
else

9
gas/config/tc-i386.c
View File

@ -10510,14 +10510,13 @@ pe_directive_secrel (int dummy ATTRIBUTE_UNUSED)
/* Handle Vector operations. */
static char *
check_VecOperations (char *op_string, char *op_end)
check_VecOperations (char *op_string)
{
const reg_entry *mask;
const char *saved;
char *end_op;
while (*op_string
&& (op_end == NULL || op_string < op_end))
while (*op_string)
{
saved = op_string;
if (*op_string == '{')
@ -11446,7 +11445,7 @@ i386_att_operand (char *operand_string)
/* Handle vector operations. */
if (*op_string == '{')
{
op_string = check_VecOperations (op_string, NULL);
op_string = check_VecOperations (op_string);
if (op_string == NULL)
return 0;
}
@ -11520,7 +11519,7 @@ i386_att_operand (char *operand_string)
vop_start = strchr (op_string, '{');
if (vop_start && vop_start < base_string)
{
if (check_VecOperations (vop_start, base_string) == NULL)
if (check_VecOperations (vop_start) == NULL)
return 0;
base_string = vop_start;
}