diff options
author | Jan Beulich <jbeulich@suse.com> | 2021-06-07 12:01:14 +0200 |
---|---|---|
committer | Jan Beulich <jbeulich@suse.com> | 2021-06-07 12:01:14 +0200 |
commit | f70c6814bb5127c155246c24cffe9d3405137c01 (patch) | |
tree | f598cb46732b0d97fdbdb4580bbcb15919181096 /gas/config/tc-i386.c | |
parent | 4e014f6ac0a2420f615a902af4370b1a45087bb6 (diff) | |
download | gdb-f70c6814bb5127c155246c24cffe9d3405137c01.zip gdb-f70c6814bb5127c155246c24cffe9d3405137c01.tar.gz gdb-f70c6814bb5127c155246c24cffe9d3405137c01.tar.bz2 |
x86: remove pointless 2nd parameter from check_VecOperations()
In the one case where non-NULL gets passed, passing NULL has the same
effect. Hence the parameter is not needed at all.
Diffstat (limited to 'gas/config/tc-i386.c')
-rw-r--r-- | gas/config/tc-i386.c | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/gas/config/tc-i386.c b/gas/config/tc-i386.c index 9e7a97f..3a59ea0 100644 --- a/gas/config/tc-i386.c +++ b/gas/config/tc-i386.c @@ -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; } |