espressif/binutils-gdb
1
0
Fork 0
mirror of https://github.com/espressif/binutils-gdb.git synced 2025-06-23 19:50:13 +08:00
Code Issues Packages Projects Releases Wiki Activity

At the point where check_VecOperands()/VEX_check_operands() get run,

Browse Source 
all other instruction attributes already matched, so any mismatch here
will tell the user more precisely what is wrong than using an eventual
(and very likely to occur) more generic error encountered on a
subsequent iteration through the template matching loop.

2012-07-31  Jan Beulich <jbeulich@suse.com>

	* config/tc-i386.c (match_template): New local variable
	'specific_error'. Set it from i.error after failed
	check_VecOperands or VEX_check_operands. Use it if set in
	preference to i.error when actually issuing disagnostic.
This commit is contained in:
Jan Beulich
2012-07-31 07:41:39 +00:00
parent 94948e646e
commit 5614d22c1d
2 changed files with 15 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
gas/ChangeLog
View File

@ -1,3 +1,10 @@
2012-07-31 Jan Beulich <jbeulich@suse.com>
* config/tc-i386.c (match_template): New local variable
'specific_error'. Set it from i.error after failed
check_VecOperands or VEX_check_operands. Use it if set in
preference to i.error when actually issuing disagnostic.
2012-07-30 Nick Clifton <nickc@redhat.com>
* po/gas.pot: Updated template.

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

@ -4015,6 +4015,7 @@ match_template (void)
unsigned int j;
unsigned int found_cpu_match;
unsigned int check_register;
enum i386_error specific_error = 0;
#if MAX_OPERANDS != 5
# error "MAX_OPERANDS must be 5."
@ -4311,13 +4312,12 @@ check_reverse:
continue;
}
/* Check if vector operands are valid. */
if (check_VecOperands (t))
continue;
/* Check if VEX operands are valid. */
if (VEX_check_operands (t))
continue;
/* Check if vector and VEX operands are valid. */
if (check_VecOperands (t) || VEX_check_operands (t))
{
specific_error = i.error;
continue;
}
/* We've found a match; break out of loop. */
break;
@ -4327,7 +4327,7 @@ check_reverse:
{
/* We found no match. */
const char *err_msg;
switch (i.error)
switch (specific_error ? specific_error : i.error)
{
default:
abort ();